Skip to content

fix(tf-checks): control terraform plan output based on show_plan flag#349

Merged
hahirwar-cd merged 1 commit intoclouddrove:masterfrom
sunnymor-cd:fix/tf-plan-summary-output
Mar 20, 2026
Merged

fix(tf-checks): control terraform plan output based on show_plan flag#349
hahirwar-cd merged 1 commit intoclouddrove:masterfrom
sunnymor-cd:fix/tf-plan-summary-output

Conversation

@sunnymor-cd
Copy link
Contributor

@sunnymor-cd sunnymor-cd commented Mar 19, 2026

📄 Terraform Plan Output Fix

📝 Description

Fixed Terraform plan output behavior to correctly respect enable_plan and show_plan inputs.
Plan output is now written to a file and displayed conditionally based on input flags.

🔧 Type of Change

  • 🐛 Bug fix

🏷️ Workflow Category

  • Terraform (tf-*)

✅ Checklist

  • Code follows project style guidelines
  • Self-review completed
  • No new warnings introduced
  • Changes tested locally and working as expected

🧪 Testing

  • enable_plan=false → Plan step is skipped
image
  • enable_plan=true + show_plan=true → Full plan output displayed
image
  • enable_plan=true + show_plan=false → Only summary line displayed
image
  • Tested without var_file input
@hahirwar-cd hahirwar-cd added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 19, 2026
Copy link
Member

@pendharevaibhav-cd pendharevaibhav-cd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hahirwar-cd hahirwar-cd changed the title fix: control terraform plan output based on show_plan flag fix(tf-checks): control terraform plan output based on show_plan flag Mar 20, 2026
@hahirwar-cd hahirwar-cd merged commit 11f26cf into clouddrove:master Mar 20, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

4 participants