-
Notifications
You must be signed in to change notification settings - Fork 78
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(eks-migration): add e2e test of migrating node groups with zero downtime #195
Conversation
5798673
to
f397aad
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.
Looks great overall
56e9c0d
to
203b78d
Compare
@lblackstone Feedback has been addressed. PTAL @lukehoban any thoughts before merging? |
34cbffa
to
372cbef
Compare
I've gone ahead and changed the test steps to leverage a bag of k/v's in |
372cbef
to
d633f6e
Compare
@lukehoban I've addressed the feedback and simplified the testing complexity using a config bag. I've also moved this code over to We've also added an extra step in this test to first scale down the 2xlarge node group ASG, before terminating it, as this has shown better test results with regards to the issue we occasionally hit. It seems that ENI's being leaked is related to the aws-cni, and hasn't yet been fully fixed. PTAL and review. |
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.
Looks great!
Resolves #40
Related:
Tutorial PR: pulumi/docs#1327
Blog Post PR: pulumi/docs#1328