terraform-google-modules / terraform-example-foundation

Shows how the CFT modules can be composed to build a secure cloud foundation
https://cloud.google.com/architecture/security-foundations
Apache License 2.0
1.18k stars 702 forks source link

fix: #1229 - fix 3-nhas symlinks #1230

Closed fmichaelobrien closed 1 month ago

fmichaelobrien commented 2 months ago

see testing on #1229

3-networks-hub-and-spoke

https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/3-networks-hub-and-spoke/envs/nonproduction/access_context.auto.tfvars

-../../access_context.auto.tfvars
+../shared/access_context.auto.tfvars

there is a 2nd symlink regression in the same directory in https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/3-networks-hub-and-spoke/envs/nonproduction/common.auto.tfvars

-../../common.auto.tfvars
+../shared/common.auto.tfvars

I'll raise an issue shortly with the fix after I test a full upstream sync of the TEF.

Screenshot 2024-05-11 at 19 09 03
daniel-cit commented 2 months ago

/gcbrun

fmichaelobrien commented 2 months ago

either of #1233 or #1230

can go in - they are the same fix

fmichaelobrien commented 2 months ago

Daniel, myself and the team - really appreciate the speed that this PR was reviewed - thank you For gcbrun - Looking into the build issues - looks quota related - will retry

fmichaelobrien commented 2 months ago

/gcbrun

daniel-cit commented 2 months ago

@fmichaelobrien could you please update the PR with the base branch?

fmichaelobrien commented 2 months ago

Will do, eta 30 min...

fmichaelobrien commented 2 months ago

updated https://github.com/CloudLandingZone/terraform-example-foundation/tree/gh1229-fix-symlinks

pulled in yesterday's https://github.com/terraform-google-modules/terraform-example-foundation/commit/87179633f1ed238575a5d76dc3b152a0a3dc945d

daniel-cit commented 2 months ago

/gcbrun

fmichaelobrien commented 1 month ago

/gcbrun

fmichaelobrien commented 1 month ago

/gcbrun

eeaton commented 1 month ago

@fmichaelobrien we've had a few concurrent PRs that have put this one out-of-date with base branch, but if you sync it I'll re-trigger the CI tests and merge now. (will hold off on a few other PR until this one passes)

fmichaelobrien commented 1 month ago

Syncing now - eta 10 min

obriensystems commented 1 month ago

sync complete - no merge conflicts on https://github.com/CloudLandingZone/terraform-example-foundation/tree/gh1229-fix-symlinks

daniel-cit commented 1 month ago

/gcbrun