Conversation
The TerragruntParser was always calling ParseWithConsolidation with consolidated=false, even when the --consolidated flag was set. This prevented the consolidation logic from ever executing, resulting in multiple separate comments instead of one consolidated comment. Changes: - Add Consolidated field to TerragruntParser struct - Update NewTerragruntParser to accept consolidated parameter - Modified Parse() to use p.Consolidated instead of hardcoded false - Update plan.go and apply.go to pass true when consolidated mode enabled - Fix test files to pass explicit false for non-consolidated tests This fixes the issue where Terragrunt run-all would post separate comments for each module instead of consolidating them into a single comment when --consolidated flag is used.
…lidated mode - Capture 'No changes' and 'Plan: X to add' summary lines - Capture resource diff details (lines with +, -, ~, #) - Capture terraform action headers - Strip terragrunt prefix before appending to buffer This fixes empty module summaries in consolidated comments.
…onsolidation - Skip duplicate consecutive lines in module buffers - Exclude root module '.' from consolidated output - Prevent 'No changes' from appearing multiple times
Member
|
@justushar I think the Title and Description should include an explanation regarding the changes made for Slack. |
j-njoto
approved these changes
Feb 5, 2026
justushar
changed the title
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
WHAT
Adds Terragrunt Parser and Terragrunt consolidated output mode for Plan operations to provide a single consolidated plan. Adds default support for Terragrunt run-all command in tfnotify.
The change also adds slack handler back into Tfnotify.
It can be used via the tfcmt configuration as follows:
Please note, the plan_title, plan_message, apply_title, apply_message as well as channel_id, and bot_token can be fetched directly from env variables.
(Write the change being made with this pull request)
WHY
For better UX in case of Terragrunt.
(Write the motivation why you submit this pull request)