schmalz/shared-actions
15
0
Fork 0
Code Issues Pull requests 2 Releases Packages Activity Actions

feat: add terraform-plan action #51

Open
Omkar.Singad wants to merge 2 commits from feat/terraform-plan into main
pull from: feat/terraform-plan
merge into: schmalz:main
Conversation 2 Commits 2 Files changed 3 +130
Omkar.Singad commented 2026-06-24 09:46:03 +00:00
Member
Copy link

Summary

This PR introduces a reusable terraform-plan action.

The goal is to add a safe preview step before Terraform changes are applied. Currently, we have validation and apply actions, but no reusable action to show what Terraform intends to change.

Why this is useful

terraform plan helps us review infrastructure changes before execution. It shows whether resources will be:

  • created
  • updated
  • deleted
  • replaced/recreated

This is especially useful for staging/dev workflows where we want to verify the impact before applying changes.

Benefits

  • Improves visibility before terraform apply
  • Helps catch unintended resource deletion or replacement
  • Makes provider/module/version updates easier to review, since their impact can be seen clearly in the plan
  • Supports safer reviews in CI/CD without modifying infrastructure
  • Keeps the workflow consistent with existing reusable Terraform actions

Important note

This action only runs terraform plan.
It does not apply any changes to infrastructure.

It is intended as a read-only safety and review step.

## Summary This PR introduces a reusable `terraform-plan` action. The goal is to add a safe preview step before Terraform changes are applied. Currently, we have validation and apply actions, but no reusable action to show what Terraform intends to change. ## Why this is useful `terraform plan` helps us review infrastructure changes before execution. It shows whether resources will be: - created - updated - *deleted* - *replaced/recreated* This is especially useful for staging/dev workflows where we want to verify the impact before applying changes. ## Benefits - Improves visibility before `terraform apply` - Helps catch unintended resource deletion or replacement - Makes provider/module/version updates easier to review, since their impact can be seen clearly in the plan - Supports safer reviews in CI/CD without modifying infrastructure - Keeps the workflow consistent with existing reusable Terraform actions ## Important note This action only runs `terraform plan`. It does not apply any changes to infrastructure. It is intended as a read-only safety and review step.
Omkar.Singad added 1 commit 2026-06-24 09:46:04 +00:00
feat: add terraform-plan action
All checks were successful
validate-shared-actions / validate-shared-actions (pull_request) Successful in 31s
Details
Aikido Security PR Check / Aikido Security Scan (pull_request) Successful in 43s
Details
0cce2c847d
requested reviews from Developers, Michael.Seele 2026-06-24 09:46:04 +00:00
Omkar.Singad self-assigned this 2026-06-24 09:46:46 +00:00
Michael.Seele requested changes 2026-06-24 10:24:19 +00:00
Michael.Seele left a comment
Owner
Copy link

A reference should be also added to the global README.md

A reference should be also added to the global README.md
terraform-plan/action.yml Outdated
@ -0,0 +29,4 @@
steps:
# Setup Terraform
- name: Setup Terraform
uses: hashicorp/setup-terraform@5e8dbf3c6d9deaf4193ca7a8fb23f2ac83bb6c85
Michael.Seele commented 2026-06-24 10:23:50 +00:00
Owner
Copy link

Please add a comment like here: https://schmalz-git.git.onstackit.cloud/schmalz/shared-actions/src/branch/main/terraform-validate/action.yml#L33

Please add a comment like here: https://schmalz-git.git.onstackit.cloud/schmalz/shared-actions/src/branch/main/terraform-validate/action.yml#L33
Omkar.Singad marked this conversation as resolved
Omkar.Singad force-pushed feat/terraform-plan from 0cce2c847d to d78669e602 2026-06-24 10:36:53 +00:00 Compare
Omkar.Singad commented 2026-06-24 10:56:24 +00:00
Author
Member
Copy link

Hi @Michael.Seele ,
Required changes has been made. Please check now.

Hi @Michael.Seele , Required changes has been made. Please check now.
Omkar.Singad force-pushed feat/terraform-plan from d78669e602 to 294b0bae45 2026-06-24 11:08:08 +00:00 Compare
All checks were successful
validate-shared-actions / validate-shared-actions (pull_request) Successful in 48s
Details
Aikido Security PR Check / Aikido Security Scan (pull_request) Successful in 58s
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/terraform-plan:feat/terraform-plan
git checkout feat/terraform-plan
Sign in to join this conversation.
Reviewers
No reviewers
Michael.Seele
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
Omkar.Singad
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: schmalz/shared-actions#51
No description provided.
Delete branch "feat/terraform-plan"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?