-
Notifications
You must be signed in to change notification settings - Fork 726
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
fix: #1229 - fix 3-nhas symlinks #1230
fix: #1229 - fix 3-nhas symlinks #1230
Conversation
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
/gcbrun |
Daniel, myself and the team - really appreciate the speed that this PR was reviewed - thank you |
/gcbrun |
@fmichaelobrien could you please update the PR with the base branch? |
Will do, eta 30 min... |
updated pulled in yesterday's 8717963 |
/gcbrun |
1 similar comment
/gcbrun |
/gcbrun |
@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) |
Syncing now - eta 10 min |
sync complete - no merge conflicts on https://github.com/CloudLandingZone/terraform-example-foundation/tree/gh1229-fix-symlinks |
/gcbrun |
see testing on #1229
3-networks-hub-and-spoke
see change in chore: Fix project labels #1107
https://github.com/terraform-google-modules/terraform-example-foundation/pull/1107/files#diff-bcfdfb477dea3f75d9d515aa61ce4f3836b223f11d6e4234122778963fbfef4a
https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/3-networks-hub-and-spoke/envs/nonproduction/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
I'll raise an issue shortly with the fix after I test a full upstream sync of the TEF.