-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Migrate EC2 to aws-sdk-go-v2 #16460
Migrate EC2 to aws-sdk-go-v2 #16460
Conversation
9626c43
to
1c24423
Compare
@rifelpet I think I am seeing a new error that prevents deleting the AWS clusters:
|
This is indeed huge, but looks great. Very nice work @rifelpet! |
daa1e86
to
78d8fb6
Compare
I was able to repro this with a very simple script. It seems like a bug in the v2 SDK given that it worked fine with the v1 SDK, but I changed the cleanup to use SecurityGroupRuleIDs instead and now it is passing 🤞🏻 |
/retest |
1 similar comment
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Ref: #16424
This largely finishes the migration to aws-sdk-go-v2. I'll update the issue with remaining steps and blockers.
This is a large PR and I did my best to break up commits into logical chunks, though each commit isnt necessarily able to compile. To get each commit to compile would have involved adding an extra
AWSCloud.EC2V2()
interface method and gradually migrating consumers to use it and then clean it up afterwards, which would have had a much wider impact./cc @hakman