Skip to content

APIGW: support DeploymentCanarySettings for AWS::ApiGateway::Deployment #12899

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 24, 2025

Conversation

bentsku
Copy link
Contributor

@bentsku bentsku commented Jul 23, 2025

Motivation

As part of some testing (for this sample, I encountered an issue with LocalStack ignoring some configuration.

It did not handle the DeploymentCanarySettings property for AWS::ApiGateway::Deployment

Changes

  • add a new test for DeploymentCanarySettings property and template
  • add support for DeploymentCanarySettings in the resource provider

@bentsku bentsku added this to the 4.7 milestone Jul 23, 2025
@bentsku bentsku self-assigned this Jul 23, 2025
@bentsku bentsku added aws:apigateway Amazon API Gateway semver: patch Non-breaking changes which can be included in patch releases labels Jul 23, 2025
Copy link

Test Results - Preflight, Unit

21 983 tests  ±0   20 249 ✅ ±0   6m 19s ⏱️ -6s
     1 suites ±0    1 734 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit d76bae7. ± Comparison against base commit 82db418.

Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 10s ⏱️ -5s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit d76bae7. ± Comparison against base commit 82db418.

Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 22m 27s ⏱️
5 297 tests 4 367 ✅ 930 💤 0 ❌
5 303 runs  4 367 ✅ 936 💤 0 ❌

Results for commit d76bae7.

Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 43m 41s ⏱️ + 1m 16s
4 938 tests +1  4 161 ✅ +1  777 💤 ±0  0 ❌ ±0 
4 940 runs  +1  4 161 ✅ +1  779 💤 ±0  0 ❌ ±0 

Results for commit d76bae7. ± Comparison against base commit 82db418.

@bentsku bentsku marked this pull request as ready for review July 24, 2025 09:02
Copy link
Contributor

@simonrw simonrw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks for submitting this PR. A nice clear change and test

Comment on lines +129 to +131
if model.get("DeploymentCanarySettings"):
params["canarySettings"] = model["DeploymentCanarySettings"]

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotta love these short code change PRs! 155 lines added, of which 3 are the implementation (2%)!

@bentsku bentsku merged commit 3949e0b into main Jul 24, 2025
44 checks passed
@bentsku bentsku deleted the apigw-fix-deployment-canary branch July 24, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws:apigateway Amazon API Gateway semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants