-
Notifications
You must be signed in to change notification settings - Fork 328
Upgrade to AWS Go SDK v2 #3554
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
Upgrade to AWS Go SDK v2 #3554
Conversation
f06ea0c to
1e335e7
Compare
0e6311f to
b1d6e9c
Compare
wolfeidau
left a comment
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.
@DrJosh9000 you may want to remove this internal/awslib/.DS_Store
51ad6c9 to
51faf87
Compare
wolfeidau
left a comment
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.
Just a few small things and questions.
51faf87 to
6a5f7ce
Compare
6a5f7ce to
2a95448
Compare
moskyb
left a comment
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.
awesome stuff, this is really fiddly work and it looks like you've done an awesome job.
wolfeidau
left a comment
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.
👍🏻
Description
Upgrade to the supported v2 of the AWS Go SDK (from v1, which is unsupported).
Context
https://linear.app/buildkite/issue/PB-861/update-to-aws-go-sdk-v2
Fixes #3533
Closes #3559
Changes
Find v2 equivalents of v1 functionality. This is mostly fine but not always possible, particularly
NewChainCredentials. Since we should be using a more standard approach to authenticating to AWS anyway (see #3559), I've changed the way that part works to be similar to #3559.Testing
go test ./...). Buildkite employees may check this if the pipeline has run automatically.go fmt ./...)Disclosures / Credits
No AI here