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.

Sign up for GitHub

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

Jump to bottom

Allow Stack targeting with workspaces #50

Merged
merged 3 commits into from
Apr 23, 2024
Merged

Allow Stack targeting with workspaces #50

merged 3 commits into from
Apr 23, 2024

Conversation

dschofie
Copy link
Member

  • Before when you targeted with --stack= and a terraform workspace was set then we would remove the Workspace setting. This now preserves the stacks settings through
  • Preserve file mode when copying over. This would allow someone to set the provider in the Stack path and then reuse it when copied over to telophasedirs
  • Add an error message if using a role that doesn't have access to the organizations API. This was failing before with an AccessDenied and not obvious to an end user

dschofie added 3 commits April 23, 2024 10:51
@dschofie
lib: wrap errors on perms for more clear messages
36441a7
@dschofie
resource account: fix --stacks target
5aab3fc 
Now we don't reuse the Stack when we match on a target to avoid losing
information like the Workspace or Role when a target is matched
@dschofie
lib terraform: preserve file mode when copying over
91bae76 
If someone wants to cache providers they will be executable and not
match the expected file view pattern. It's best to preserve permissions
when possible
@dschofie dschofie merged commit bd784d6 into main Apr 23, 2024
1 check passed
@dschofie dschofie deleted the dschofie/stack_and_workspaces branch April 23, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dschofie