-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
BigQuery Dataset default CMEK encryption #2226
BigQuery Dataset default CMEK encryption #2226
Conversation
Hi! I'm the modular magician, I work on Magic Modules. Pull request statusesNew Pull RequestsI built this PR into one or more new PRs on other repositories, and when those are closed, this PR will also be merged and closed. |
I'm going to be leaving pretty soon for vacation. Sam, mind taking care of this review? |
Any feedback on this? |
Oh gosh, sorry I dropped the ball on this. Will run the tests and take a look now |
@drebes Looks good! Sorry getting to this took so long. Can you amend this commit and push it again? I'll need to re-trigger generation to update all the downstreams, then I can merge it |
Hi! I'm the modular magician, I work on Magic Modules. Pull request statusesterraform-provider-google-beta already has an open PR. New Pull RequestsI built this PR into one or more new PRs on other repositories, and when those are closed, this PR will also be merged and closed. |
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.
Thanks for the contribution!
Tracked submodules are build/terraform-beta build/terraform-mapper build/terraform build/ansible build/inspec.
Release Note for Downstream PRs (will be copied)