Add terragrunt run-all helpers and refactor helm module #1601
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This change introduces new helper functions for terragrunt run-all operations and removes unnecessary empty document handling from the helm module. Changes: - Add ApplyAll/ApplyAllE functions to run terragrunt apply with run-all - Add DestroyAll/DestroyAllE functions to run terragrunt destroy with run-all - Add PlanAllExitCode/PlanAllExitCodeE to get detailed exit codes from plan operations - Add comprehensive test coverage for the new terragrunt helpers - Remove empty YAML document handling from helm/template.go (no longer needed) - Remove corresponding empty document tests from helm/template_test.go
Add deprecation warnings to all terragrunt stack-related functions: - TgStackRun/TgStackRunE: Recommend using ApplyAll/ApplyAllE - TgStackClean/TgStackCleanE: Recommend using DestroyAll/DestroyAllE - TgStackGenerate/TgStackGenerateE: Recommend using PlanAllExitCode/ApplyAll - TgOutput/TgOutputE/TgOutputJson/TgOutputJsonE: Recommend using terraform.Output methods The 'stack' commands are deprecated in Terragrunt and users should migrate to the newer run-all commands or individual module operations.
Critical fixes: - Rename getExitCodeForTerraformCommandE to getExitCodeForTerragruntCommandE for clarity (it runs terragrunt, not terraform) Warning fixes: - Rename TestApplyAllTgError to TestApplyAllNoError (test uses no-error fixture) - Rename TestDestroyAllTgError to TestDestroyAllNoError and add assertions - Add proper assertion for destroy output to ensure it completes successfully Minor improvements: - Update function comments to explicitly specify 'run-all' commands - Standardize comment formatting (returns vs return, exit code consistency)
denis256
approved these changes
Oct 15, 2025
|
Hi @james00012! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR introduces new helper functions for terragrunt
run-alloperations and refactors the helm module by removing unnecessary empty document handling.Changes
New Terragrunt Helpers
terragrunt run-all applywith auto-approveterragrunt run-all destroywith auto-approveterragrunt run-all planand capture detailed exit codes (0, 1, or 2)Helm Module Refactoring
modules/helm/template.gomodules/helm/template_test.goTest Plan
Files Changed
modules/terragrunt/apply.go(new)modules/terragrunt/apply_test.go(new)modules/terragrunt/destroy.go(new)modules/terragrunt/destroy_test.go(new)modules/terragrunt/plan.go(new)modules/terragrunt/plan_test.go(new)modules/terragrunt/cmd.go(updated)modules/helm/template.go(updated)modules/helm/template_test.go(updated)