Skip to content
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

externalconn,amazon: support s3 KMS in External Connecetions #86402

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

adityamaru
Copy link
Contributor

@adityamaru adityamaru commented Aug 18, 2022

Informs: #84228

Release note (sql change): Users can now
CREATE EXTERNAL CONNECTION to represent an aws-kms
scheme that represents an AWS KMS resource.

Release justification: low risk change to new functionality to register s3 KMS as a supported External Connection

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@adityamaru adityamaru force-pushed the s3-kms branch 2 times, most recently from 2dd9389 to 8da6b9d Compare August 19, 2022 16:44
@adityamaru adityamaru marked this pull request as ready for review August 22, 2022 13:54
@adityamaru adityamaru requested review from a team, rhu713 and benbardin and removed request for a team and rhu713 August 22, 2022 13:54
Informs: cockroachdb#84228

Release note (sql change): Users can now
`CREATE EXTERNAL CONNECTION` to represent an `aws-kms`
scheme that represents an AWS KMS resource.
@adityamaru
Copy link
Contributor Author

adityamaru commented Aug 22, 2022

Passing cloud unit test run - https://teamcity.cockroachdb.com/viewLog.html?buildId=6192738&buildTypeId=Cockroach_Nightlies_CloudUnitTests

The flake is similar to #86616.

bors r=benbardin

@craig
Copy link
Contributor

craig bot commented Aug 23, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Aug 23, 2022

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants