-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Promote Firestore database cmek support to GA #11387
Promote Firestore database cmek support to GA #11387
Conversation
Hello! I am a robot. Tests will require approval from a repository maintainer to run. @trodge, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 20 Click here to see the affected service packages
Tests were added that are GA-only additions and require manual runs:
View the build log |
Hi @trodge Could you please help to review my PR to promote Firestore cmek support for Terraform to GA branch please? Thank you! |
Taking a look at it now. Would it be possible for you to share test logs with me for the GA run of |
Sure! Here is the test log:
|
@trodge This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
From searching the logs I wasn't able to find any Normally I'd expect the test for an immutable field to create and then destroy the resource. |
Oh sorry some of the log output has been cut, here's the POST request where it tried to create CMEK database:
|
Promote Firestore database cmek support to GA:
cmek_config
togoogle_firestore_database
to support creating a Firestore CMEK database.Self check:
Release Note Template for Downstream PRs (will be copied)