-
Notifications
You must be signed in to change notification settings - Fork 398
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
Add check_mode support to ec2_asg #1033
Add check_mode support to ec2_asg #1033
Conversation
recheck |
8c4060b
to
692b474
Compare
LGTM, can you also add a changelog fragment please? |
a197021
to
cbdd8fd
Compare
could you add more check_mode tests? for general operations like enabling/disabling metrics collection, but also idempotence check_mode tests as well. afaik each test case should cover check_mode, action, check_mode idempotence, idempotence |
With the general behavior of the API, and the amount of time it takes to adjust the ASG configuration, it would need to add more Unless others have different opinion, then we can try adding more tests where appropriate. |
Can you add waiters? |
Backport to stable-2: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 13a20d4 on top of patchback/backports/stable-2/13a20d48e2cc1afe2434243bc41d8b8e95faddfb/pr-1033 Backporting merged PR #1033 into main
🤖 @patchback |
Backport to stable-3: 💚 backport PR created✅ Backport PR branch: Backported as #1046 🤖 @patchback |
Add check_mode support to ec2_asg SUMMARY Added check_mode support to ec2_asg. CI failure could be resolved by #1036 ISSUE TYPE Feature Pull Request COMPONENT NAME ec2_asg Reviewed-by: Markus Bergholz <[email protected]> Reviewed-by: Jill R <None> (cherry picked from commit 13a20d4)
[PR #1033/13a20d48 backport][stable-3] Add check_mode support to ec2_asg This is a backport of PR #1033 as merged into main (13a20d4). SUMMARY Added check_mode support to ec2_asg. CI failure could be resolved by #1036 ISSUE TYPE Feature Pull Request COMPONENT NAME ec2_asg Reviewed-by: Markus Bergholz <[email protected]>
SUMMARY
Added check_mode support to
ec2_asg
.ISSUE TYPE
COMPONENT NAME
ec2_asg