Skip to content
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

Merged

Conversation

fmichaelobrien
Copy link
Contributor

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

@fmichaelobrien fmichaelobrien changed the title #1229 - fix 3-nhas symlinks fix: #1229 - fix 3-nhas symlinks May 13, 2024
Copy link
Contributor

@daniel-cit daniel-cit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daniel-cit
Copy link
Contributor

/gcbrun

@fmichaelobrien
Copy link
Contributor Author

either of #1233
or #1230

can go in - they are the same fix

@fmichaelobrien
Copy link
Contributor Author

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
Copy link
Contributor Author

/gcbrun

@daniel-cit
Copy link
Contributor

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

@fmichaelobrien
Copy link
Contributor Author

Will do, eta 30 min...

@fmichaelobrien
Copy link
Contributor Author

@daniel-cit
Copy link
Contributor

/gcbrun

1 similar comment
@fmichaelobrien
Copy link
Contributor Author

/gcbrun

@fmichaelobrien
Copy link
Contributor Author

/gcbrun

@eeaton
Copy link
Collaborator

eeaton commented May 29, 2024

@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)

@eeaton eeaton enabled auto-merge (squash) May 29, 2024 17:06
@fmichaelobrien
Copy link
Contributor Author

Syncing now - eta 10 min

@obriensystems
Copy link
Contributor

@daniel-cit
Copy link
Contributor

/gcbrun

@eeaton eeaton merged commit 05a2308 into terraform-google-modules:master May 29, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants