-
Notifications
You must be signed in to change notification settings - Fork 114
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
Initial plumbing to support Encrypt, and KeyID (KMS Encryption) #166
Initial plumbing to support Encrypt, and KeyID (KMS Encryption) #166
Conversation
/hold i think we need to have some discussions about exactly how deep we want to get into the s3 bucket management business. |
@bparees Do I need to find a way to automate testing this KMS setup? |
i think it's sufficient to confirm we propagate the config to the registry successfully, we don't need to actually do a full bucket setup and test. |
/hold cancel |
AWS limit issues /retest |
@coreydaley squash and lgtm |
@bparees squashed. |
/lgtm |
/retest |
had to rename testframework to framework in aws_test.go, weird |
/retest |
6 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
Forgot that these only exist if set now, fixed in current push:
|
Encrypt - Enable or disable KMS encryption on S3 KeyID - The KMS Key ID for the S3 bucket fixes #161
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees, coreydaley 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 |
@bparees thanks, i was going to tag you after the tests passed |
/retest |
3 similar comments
/retest |
/retest |
/retest |
Initial plumbing to support the following options on S3:
fixes #161