-
Notifications
You must be signed in to change notification settings - Fork 153
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
S3 assume role support #290
Conversation
It will be used to signal S3 provider to switch roles before establishing connection.
if err != nil { | ||
return nil, errors.Wrap(err, "Failed to get role credentials") | ||
} | ||
return &awsCreds{ |
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.
note: not for this PR, but we should consider using the AWS type credential.Value
@@ -48,9 +49,11 @@ type ObjectStoreProviderSuite struct { | |||
const ( | |||
testBucketName = "kio-store-tests" | |||
testRegionS3 = "us-west-2" | |||
testRole = "" |
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.
We need to create a role for Kanister test purpose, and make it settable by environment variables.
Closing this one in favor of #292 |
Change Overview
Add Kanister to support assuming roles for S3 operations.
Pull request type
Please check the type of change your PR introduces:
Test Plan