-
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
Bugfix/fix 4-projects issues #374
Bugfix/fix 4-projects issues #374
Conversation
/gcbrun |
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 @renato-rudnicki
Can you update the readme with the required steps as well
https://github.com/terraform-google-modules/terraform-example-foundation/tree/develop/4-projects#setup-to-run-via-cloud-build
Increasing lifetime field in providers.tf from 900s to 1200s due timeout during build tests. |
/gcbrun |
provider = google.impersonate | ||
target_service_account = local.tf_sa | ||
scopes = ["userinfo-email", "cloud-platform"] | ||
lifetime = "600s" |
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.
Let's keep this uniform here and throughout
lifetime = "600s" | |
lifetime = "1200s" |
Updating lifetime to 1200ms in shared/providers.tf in Business Unit 1 and 2.
|
build
checking the previous steps, the reason for project 580269401305 not being in the perimeter was a concurrente update of the service perimeter by the two dev projects, bu1 and bu2. it is necessary to add an explicitly dependency between the modules in the fixture for step 4-projects so that the bu2 projects wait for their pair in bu1 to run before updating the service perimeter. This error does not happens in the cloud build deploy because the shell script serializes the execution of the two projects for the same environment. |
@daniel-cit Thanks for investigating, I do see the converge worked fine For serializing you may have to do something like 85dec25 |
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.
LGTM
* Fix issue 350 - common.auto.tfvars link pointing to an example file * Fix issue 351 - Missing backend.tf file and provider.tf from shared folder * Adding versions.tf * Adding versions.tf for BU 1 and 2 * Fixing access_context * Adding extra steps for Business Unit 1 and 2 * Updating Readme.md and adding business_unit example files * Adding simbolic links for business_unit 1 and 2 * Increasing providers.tf from 900s to 1200s * Updating lifetime to 1200ms in shared/providers.tf
* Fix issue 350 - common.auto.tfvars link pointing to an example file * Fix issue 351 - Missing backend.tf file and provider.tf from shared folder * Adding versions.tf * Adding versions.tf for BU 1 and 2 * Fixing access_context * Adding extra steps for Business Unit 1 and 2 * Updating Readme.md and adding business_unit example files * Adding simbolic links for business_unit 1 and 2 * Increasing providers.tf from 900s to 1200s * Updating lifetime to 1200ms in shared/providers.tf
* Fix issue 350 - common.auto.tfvars link pointing to an example file * Fix issue 351 - Missing backend.tf file and provider.tf from shared folder * Adding versions.tf * Adding versions.tf for BU 1 and 2 * Fixing access_context * Adding extra steps for Business Unit 1 and 2 * Updating Readme.md and adding business_unit example files * Adding simbolic links for business_unit 1 and 2 * Increasing providers.tf from 900s to 1200s * Updating lifetime to 1200ms in shared/providers.tf
* Fix issue 350 - common.auto.tfvars link pointing to an example file * Fix issue 351 - Missing backend.tf file and provider.tf from shared folder * Adding versions.tf * Adding versions.tf for BU 1 and 2 * Fixing access_context * Adding extra steps for Business Unit 1 and 2 * Updating Readme.md and adding business_unit example files * Adding simbolic links for business_unit 1 and 2 * Increasing providers.tf from 900s to 1200s * Updating lifetime to 1200ms in shared/providers.tf
This PR contains the fix for the following issues: