-
Notifications
You must be signed in to change notification settings - Fork 696
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
4-projects GCS CMEK module #346
4-projects GCS CMEK module #346
Conversation
Need to investigate why Checks fail, would love a draft review and any comments. |
…/terraform-example-foundation into feature/cmek-gcs-test-rupin
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 PR @rutalreja-deloitte
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.
Looks like the integration tests are failing at 4-projects
Could you take a look?
Also looks like there is a merge conflict, you can rebase on develop.
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.
…/terraform-example-foundation into feature/cmek-gcs-test-rupin
…/terraform-example-foundation into feature/cmek-gcs-test-rupin
@bharathkkb CMEK - MultiEnv 3a235ac was failing; terraform was trying to create the GCS bucket before the IAM policy for Encrypter/Decrypter was set on the KMS Key. |
Thanks for investigating @rutalreja-deloitte |
* 4-projects GCS CMEK module * 4-projects GCS CMEK module - Quick Fix * 4-projects GCS CMEK module - Lint * 4-projects GCS CMEK module - Validate * 4-projects GCS CMEK module - README * 4-projects GCS CMEK module - README2 + KMS version * 4-projects GCS CMEK module - Bug Fix 2 * 4-projects GCS CMEK module - Requested Changes * 4-projects GCS CMEK module - Bug Fix 3 * 4-projects GCS CMEK module - Key Destroy Support * 4-projects GCS CMEK module - Key Map Update * 4-projects GCS CMEK module - Variables * 4-projects GCS CMEK module - Final * CMEK - MultiEnv * CMEK Multi-Env Explicit Depends on
* 4-projects GCS CMEK module * 4-projects GCS CMEK module - Quick Fix * 4-projects GCS CMEK module - Lint * 4-projects GCS CMEK module - Validate * 4-projects GCS CMEK module - README * 4-projects GCS CMEK module - README2 + KMS version * 4-projects GCS CMEK module - Bug Fix 2 * 4-projects GCS CMEK module - Requested Changes * 4-projects GCS CMEK module - Bug Fix 3 * 4-projects GCS CMEK module - Key Destroy Support * 4-projects GCS CMEK module - Key Map Update * 4-projects GCS CMEK module - Variables * 4-projects GCS CMEK module - Final * CMEK - MultiEnv * CMEK Multi-Env Explicit Depends on
* 4-projects GCS CMEK module * 4-projects GCS CMEK module - Quick Fix * 4-projects GCS CMEK module - Lint * 4-projects GCS CMEK module - Validate * 4-projects GCS CMEK module - README * 4-projects GCS CMEK module - README2 + KMS version * 4-projects GCS CMEK module - Bug Fix 2 * 4-projects GCS CMEK module - Requested Changes * 4-projects GCS CMEK module - Bug Fix 3 * 4-projects GCS CMEK module - Key Destroy Support * 4-projects GCS CMEK module - Key Map Update * 4-projects GCS CMEK module - Variables * 4-projects GCS CMEK module - Final * CMEK - MultiEnv * CMEK Multi-Env Explicit Depends on
* 4-projects GCS CMEK module * 4-projects GCS CMEK module - Quick Fix * 4-projects GCS CMEK module - Lint * 4-projects GCS CMEK module - Validate * 4-projects GCS CMEK module - README * 4-projects GCS CMEK module - README2 + KMS version * 4-projects GCS CMEK module - Bug Fix 2 * 4-projects GCS CMEK module - Requested Changes * 4-projects GCS CMEK module - Bug Fix 3 * 4-projects GCS CMEK module - Key Destroy Support * 4-projects GCS CMEK module - Key Map Update * 4-projects GCS CMEK module - Variables * 4-projects GCS CMEK module - Final * CMEK - MultiEnv * CMEK Multi-Env Explicit Depends on
CMEK Module first review submission
Tests to follows in next PR