-
Notifications
You must be signed in to change notification settings - Fork 57
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
Replace/revert-back CB/CSR options as we add local terraform and ado (399) options - to be able to use CB/CSR as a current option while we retrofit #421
Comments
Do TEF alignment after 421 #387 keep
ChangesAdjust
Help with upstream sync
changes 0-bootstrap to 3-networks-dual-svpc
|
part 2 remaining part
|
Testing on existing org - no TEF run before - oldev downgrade terraform and prep project
terraform init / plan / apply
total time actual: 50 min to 1821 1731 apply - 1 min and group eventually consistent error
wait 5 min 1733-1738 plan / apply 2 - 5 min later 1739
forgot biling_id 3 - plan / apply 1743
1759 In past deployments - missed adding cb service enablement
4 - plan / apply 1800
wait 5 min for cloudbuild enablement to propagate 1804-18095 - plan / apply 1809
1810 idempotent error - either full terraform destroy - or orphan/rename the resourceprivate-pool-rtbh northamerica-northeast1 e2-medium 100 GB vpc-b-cbpools 6 - plan/apply after cb worker pool delete - 1817
1819 |
continue 0-bootstrap
check cloud build 1853
|
#421 - prepare main for dual CB/CSR and terraform local deployment modes
#421 - main branch defaults to CB/CSR
Additional local/cb retrofit for #423
ok |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days |
branch: https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/tree/gh421-cb-enable
The current main is a partial local terraform retrofit that has a pending pr to complete the addition of local terraform capability - see workaround branch for now in #420
For example, in 0-bootstrap, the previous yaml for CB - cb.tf is commented as https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/blob/main/0-bootstrap/cb.tf.dont_use
Also the variables and output tf files need to be retrofitted back with the CB/CSR sections commented - in prep for use - like we do for github/githlab/tc
Either as part of #399 or earlier - put back the build so that cb.tf is listed as cb.tf.example with full readme instructions in addition to the variables/output changes.
The text was updated successfully, but these errors were encountered: