[PR #301] [MERGED] 293 terraform workflow improvement #371

New issue

Closed

opened 2026-02-27 19:17:56 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:17:56 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/301
Author: @ekosfin
Created: 4/22/2024
Status: ✅ Merged
Merged: 4/22/2024
Merged by: @ekosfin

Base: master ← Head: 293-terraform-workflow-improvement

📝 Commits (5)

adc14cf Modified workflow to output plan to pr comment
84344a2 Added terraform apply step
9777d57 removed checkout ref as it interferes with forked pull requests
85a747c Added path to upload
183b545 main to master

📊 Changes

1 file changed (+163 additions, -74 deletions)

View changed files

📝 .github/workflows/terraform.yml (+163 -74)

📄 Description

Did a bit of an rework on the terraform workflow. Now it does not commit to the branch, but fails the workflow if the formatting is wrong. Also, it should now comment on the pull request with the plan for review.

Next steps would be to set up OICD and two IAM roles that correspond with plan and apply permissions.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/EddieTheCubeHead/Stagnum/pull/301 **Author:** [@ekosfin](https://github.com/ekosfin) **Created:** 4/22/2024 **Status:** ✅ Merged **Merged:** 4/22/2024 **Merged by:** [@ekosfin](https://github.com/ekosfin) **Base:** `master` ← **Head:** `293-terraform-workflow-improvement` --- ### 📝 Commits (5) - [`adc14cf`](https://github.com/EddieTheCubeHead/Stagnum/commit/adc14cf6152b3e0f4da0fa8055c7b387c878bb50) Modified workflow to output plan to pr comment - [`84344a2`](https://github.com/EddieTheCubeHead/Stagnum/commit/84344a2f79a5d8ab88738d778e6cc2e540fa58d6) Added terraform apply step - [`9777d57`](https://github.com/EddieTheCubeHead/Stagnum/commit/9777d57bf851a7e33e7b7d9a73fd2d81e4700904) removed checkout ref as it interferes with forked pull requests - [`85a747c`](https://github.com/EddieTheCubeHead/Stagnum/commit/85a747cba02ae234c8d970b549864b17a2eabe82) Added path to upload - [`183b545`](https://github.com/EddieTheCubeHead/Stagnum/commit/183b5451d79beaa082c6a99e6986dbf4f4770aac) main to master ### 📊 Changes **1 file changed** (+163 additions, -74 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/terraform.yml` (+163 -74) </details> ### 📄 Description Did a bit of an rework on the terraform workflow. Now it does not commit to the branch, but fails the workflow if the formatting is wrong. Also, it should now comment on the pull request with the plan for review. Next steps would be to set up OICD and two IAM roles that correspond with plan and apply permissions. Related: https://github.com/EddieTheCubeHead/Stagnum/issues/293 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>