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

apply-all should error if TERRAGRUNT_SOURCE is specified #1339

Closed
yinzara opened this issue Sep 12, 2020 · 1 comment
Closed

apply-all should error if TERRAGRUNT_SOURCE is specified #1339

yinzara opened this issue Sep 12, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@yinzara
Copy link

yinzara commented Sep 12, 2020

I caused a MAJOR production outage across a set of tenants because I accidentally had the TERRAGRUNT_SOURCE environment variable set when I performed an "apply-all" across our tenants. This caused more than 800 resources across 6 customers to be deleted because TG attempt to apply the source from TERRAGRUNT_SOURCE against a set of terragrunt.hcl files that were for different modules.

TERRAGRUNT_SOURCE is incompatible with apply-all unless all modules being applied are for the same source. apply-all should have prevented this from happening either by erroring or prompting me to confirm specifically about TERRAGRUNT_SOURCE.

@brikis98 brikis98 added enhancement New feature or request help wanted labels Sep 13, 2020
@brikis98
Copy link
Member

Sorry to hear you hit this issue. That sounds really painful 😕

FWIW, we use TERRAGRUNT_SOURCE quite regularly with apply-all, especially when spinning up totally new environments. That said, there are a number of limitations / gotchas with TERRAGRUNT_SOURCE, and we've captured a proposal for a better replacement in #1138. Let's continue the discussion in there. I'll close this issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants