-
Notifications
You must be signed in to change notification settings - Fork 1.2k
test: Fix flaky disable dry-run test #8421
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
test: Fix flaky disable dry-run test #8421
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
|
Preview deployment ready! Preview URL: https://pr-8421.d18coufmbnnaag.amplifyapp.com Built from commit |
Pull Request Test Coverage Report for Build 17304487769Details
💛 - Coveralls |
2b8dbb5 to
ad3916c
Compare
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.
/karpenter snapshot
|
Snapshot successfully published to |
d0290a0 to
be9c94c
Compare
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.
/karpenter snapshot
be9c94c to
1bc4134
Compare
|
Snapshot successfully published to |
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.
/karpenter snapshot
|
Snapshot successfully published to |
1bc4134 to
6a576c3
Compare
6a576c3 to
0233cb6
Compare
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.
/karpenter snapshot
|
Snapshot successfully published to |
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.
/karpenter snapshot
Fixes #N/A
Description
Update the disable dry-run test to ensure that we properly wait for IAM to ensure that we have the proper permissions at the end of the test after we remove the role policy that blocks permissions for the dry-run validation to succeed.
How was this change tested?
make presubmit/karpenter snapshotLocally testing the "should succeed EC2NodeClass validation when dry run validation is disabled" test with the updated cleanup logic
Does this change impact docs?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.