Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Null vars file cleanup #2672

Merged
merged 3 commits into from
Aug 17, 2023
Merged

Null vars file cleanup #2672

merged 3 commits into from
Aug 17, 2023

Conversation

denis256
Copy link
Member

@denis256 denis256 commented Aug 17, 2023

Description

Included changes:

  • skip generation of .terragrunt-null-vars.auto.tfvars.json if there are no null values
  • removal of .terragrunt-null-vars.auto.tfvars.json after terraform execution

Fixes #2669.
Fixes #2670

TODOs

Read the Gruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].

Updated Terragrunt to not generate .terragrunt-null-vars.auto.tfvars.json if is not required.

Migration Guide

@denis256 denis256 changed the title [WIP] Added step to clean null var files Null vars file cleanup Aug 17, 2023
@denis256 denis256 marked this pull request as ready for review August 17, 2023 19:41
@denis256 denis256 merged commit 79a54ec into master Aug 17, 2023
@denis256 denis256 deleted the bug/vars-generation-skip-2669 branch August 18, 2023 19:31
hugorut pushed a commit to infracost/terragrunt that referenced this pull request Oct 10, 2023
* Added step to clean null var files

* Cleanup file creation

* Add to track removal of null values json file
hugorut pushed a commit to infracost/terragrunt that referenced this pull request Oct 10, 2023
* Added step to clean null var files

* Cleanup file creation

* Add to track removal of null values json file
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants