-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
Remove CodeDeploy alarm limit #39971
Conversation
Community NoteVoting for Prioritization
For Submitters
|
These pages are generated automatically by a scheduled internal process.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
% make testacc PKG=deploy TESTS=TestAccDeployDeploymentGroup_
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/deploy/... -v -count 1 -parallel 20 -run='TestAccDeployDeploymentGroup_' -timeout 360m
--- PASS: TestAccDeployDeploymentGroup_OutdatedInstancesStrategy_update (41.89s)
=== CONT TestAccDeployDeploymentGroup_LoadBalancerInfo_delete
--- PASS: TestAccDeployDeploymentGroup_TermationHook_disabled (46.23s)
=== CONT TestAccDeployDeploymentGroup_LoadBalancerInfo_update
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfo_create (56.17s)
=== CONT TestAccDeployDeploymentGroup_tags
--- PASS: TestAccDeployDeploymentGroup_TermationHook_enabled (66.13s)
=== CONT TestAccDeployDeploymentGroup_AutoRollback_update
--- PASS: TestAccDeployDeploymentGroup_DeploymentStyle_delete (74.41s)
=== CONT TestAccDeployDeploymentGroup_AutoRollback_create
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfoTargetGroupInfo_update (74.73s)
=== CONT TestAccDeployDeploymentGroup_Trigger_multiple
--- PASS: TestAccDeployDeploymentGroup_BlueGreenDeployment_update (75.26s)
=== CONT TestAccDeployDeploymentGroup_Trigger_basic
--- PASS: TestAccDeployDeploymentGroup_OutdatedInstancesStrategy_ignore (75.28s)
=== CONT TestAccDeployDeploymentGroup_Alarm_disable
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfoTargetGroupInfo_create (76.62s)
=== CONT TestAccDeployDeploymentGroup_DeploymentStyle_update
--- PASS: TestAccDeployDeploymentGroup_InPlaceDeploymentWithTrafficControl_update (80.19s)
=== CONT TestAccDeployDeploymentGroup_DeploymentStyle_create
--- PASS: TestAccDeployDeploymentGroup_InPlaceDeploymentWithTrafficControl_create (81.31s)
=== CONT TestAccDeployDeploymentGroup_DeploymentStyle_default
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfoTargetGroupInfo_multiple (82.04s)
=== CONT TestAccDeployDeploymentGroup_Alarm_update
--- PASS: TestAccDeployDeploymentGroup_BlueGreenDeployment_delete (94.62s)
=== CONT TestAccDeployDeploymentGroup_Alarm_delete
--- PASS: TestAccDeployDeploymentGroup_AutoRollback_delete (96.83s)
=== CONT TestAccDeployDeploymentGroup_disappears
--- PASS: TestAccDeployDeploymentGroup_BlueGreenDeployment_complete (97.50s)
=== CONT TestAccDeployDeploymentGroup_Disappears_app
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfoTargetGroupInfo_delete (98.27s)
=== CONT TestAccDeployDeploymentGroup_onPremiseTag
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfo_delete (62.87s)
=== CONT TestAccDeployDeploymentGroup_Alarm_create
--- PASS: TestAccDeployDeploymentGroup_basic (104.78s)
=== CONT TestAccDeployDeploymentGroup_AutoRollback_disable
--- PASS: TestAccDeployDeploymentGroup_LoadBalancerInfo_update (61.97s)
=== CONT TestAccDeployDeploymentGroup_Basic_tagSet
--- PASS: TestAccDeployDeploymentGroup_DeploymentStyle_create (31.69s)
--- PASS: TestAccDeployDeploymentGroup_DeploymentStyle_default (31.43s)
--- PASS: TestAccDeployDeploymentGroup_AutoRollback_update (48.05s)
--- PASS: TestAccDeployDeploymentGroup_tags (64.95s)
--- PASS: TestAccDeployDeploymentGroup_onPremiseTag (23.40s)
--- PASS: TestAccDeployDeploymentGroup_Disappears_app (27.85s)
--- PASS: TestAccDeployDeploymentGroup_disappears (30.85s)
--- PASS: TestAccDeployDeploymentGroup_AutoRollback_disable (35.04s)
--- PASS: TestAccDeployDeploymentGroup_Alarm_create (36.02s)
--- PASS: TestAccDeployDeploymentGroup_AutoRollback_create (68.47s)
--- PASS: TestAccDeployDeploymentGroup_Alarm_update (69.92s)
--- PASS: TestAccDeployDeploymentGroup_BlueGreenDeployment_create (151.96s)
--- PASS: TestAccDeployDeploymentGroup_Basic_tagSet (45.54s)
--- PASS: TestAccDeployDeploymentGroup_Trigger_multiple (80.06s)
--- PASS: TestAccDeployDeploymentGroup_Trigger_basic (81.34s)
--- PASS: TestAccDeployDeploymentGroup_DeploymentStyle_update (86.68s)
--- PASS: TestAccDeployDeploymentGroup_Alarm_disable (89.81s)
--- PASS: TestAccDeployDeploymentGroup_Alarm_delete (77.91s)
--- PASS: TestAccDeployDeploymentGroup_BlueGreenDeployment_updateWithASG (191.06s)
--- PASS: TestAccDeployDeploymentGroup_ECS_blueGreen (360.96s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/deploy 366.627s
Thanks for your contribution, @AnitaErnszt! 👍 |
This functionality has been released in v5.75.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
Removing the alarm limit from CodeDeploy.
The provider currently only support max 10 alarms, a limit which was set 8 years ago (ref)
AWS now supports 50 alarms per deployment group, and this limit is adjustable
Relations
Haven't opened issue, but happy to if needed, this is currently a blocker for us
Output from Acceptance Testing