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

[BUG] terragrunt apply uses mock outputs #1998

Open
dudicoco opened this issue Jan 27, 2022 · 3 comments · May be fixed by #2655
Open

[BUG] terragrunt apply uses mock outputs #1998

dudicoco opened this issue Jan 27, 2022 · 3 comments · May be fixed by #2655
Assignees
@denis256
Labels
bug Something isn't working p:needs triage Needs to be processed by maintainer and issue type / priority added

Comments

@dudicoco
Copy link

When running terragrunt apply on a terragrunt.hcl which has a dependency block with mock_outputs, if the dependency does not have any outputs terragrunt uses the mock outputs values on apply instead of throwing an error.

I believe this behavior did not exist in the past and terragrunt threw an error if no outputs are found.
@denis256 denis256 added bug Something isn't working p:needs triage Needs to be processed by maintainer and issue type / priority added labels Jan 30, 2022
@dennislapchenko
Copy link

Have noticed exactly the same behaviour

@dudicoco
Copy link
Author

@denis256 @yorinasub17 @brikis98 can you please advise?

@denis256
Copy link
Member

Hi,
will be helpful to share how dependency is defined and how terragrunt was invoked(run or run-all).

Was a set value for mock_outputs_allowed_terraform_commands?, or skip_outputs was enabled?

Try to use on dependency definition:

mock_outputs_allowed_terraform_commands = ["validate"]

References:

https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/#dependency

@denis256 denis256 moved this to To do in Terragrunt Roadmap Aug 2, 2023
@denis256 denis256 self-assigned this Aug 2, 2023
@denis256 denis256 moved this from To do to In progress in Terragrunt Roadmap Aug 3, 2023
@denis256 denis256 linked a pull request Aug 3, 2023 that will close this issue
Draft
4 tasks
@denis256 denis256 moved this from In progress to To do in Terragrunt Roadmap Sep 15, 2023
@denis256 denis256 mentioned this issue Jan 28, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@denis256 denis256

Labels
bug Something isn't working p:needs triage Needs to be processed by maintainer and issue type / priority added
Projects
Terragrunt Roadmap
Status: To do
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[WIP] Updated handling of empty mock command list gruntwork-io/terragrunt
3 participants
@denis256 @dennislapchenko @dudicoco