Skip to content

Support plan-diff#77

Merged
Igor Rodionov (goruha) merged 124 commits into
mainfrom
plan-diff
Aug 14, 2025
Merged

Support plan-diff#77
Igor Rodionov (goruha) merged 124 commits into
mainfrom
plan-diff

Conversation

@goruha

@goruha Igor Rodionov (goruha) commented Aug 5, 2025

Copy link
Copy Markdown
Member

what

  • Use atmos terraform plan-diff to ensure stored plan is actual
  • Added tests for
    • Basic apply usage
    • Apply with storage disabled
    • Apply with atmos pro
    • Apply when plan-diff has changes
    • Apply when plan generation fail
  • Updated README with migration plan for new v5 action
  • Added Github Action summaries for cases plan-diff found changes and plan generation failed
  • Updated terraform-docs to v0.20.0

why

  • Guarantee that the plan we saw on GitOps plan stage would be the same one we apply.
  • Improve test coverage to be confident in GHA behaviour
  • Provide documentation to users about the changes
  • Provide a nice feedback message in the GitHub UI for the action
  • Get the latest bug fixes

@goruha Igor Rodionov (goruha) added the major Breaking changes (or first stable release) label Aug 10, 2025
This was referenced Aug 10, 2025

@Benbentwo Ben (Benbentwo) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good but have some concerns about 60+ line bash scripts

Comment thread action.yml
@goruha Igor Rodionov (goruha) merged commit 267eec5 into main Aug 14, 2025
19 of 23 checks passed
@goruha Igor Rodionov (goruha) deleted the plan-diff branch August 14, 2025 13:26
@github-actions

Copy link
Copy Markdown

These changes were released in v5.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major Breaking changes (or first stable release)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants