Skip to content

wip: add terraform action#15

Draft
tosuke wants to merge 1 commit into
mainfrom
wip-tfaction
Draft

wip: add terraform action#15
tosuke wants to merge 1 commit into
mainfrom
wip-tfaction

Conversation

@tosuke

@tosuke tosuke commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

供養

戦略

  • PR 上で plan を生成し artifact に (ロックがないので refresh しない)
  • Merge Queue の並列数を1に制限した上で apply を実行する
  • apply に成功したら main ブランチにマージ、そうでなければ plan を再計算して PR に戻す

懸念

  • シークレット管理
    • plan のためには state の読み取り権限が必要だが、公開リポジトリでこれは難しい
  • 管理コスト
    • それなりに複雑なワークフローが必要になる

戦略
- PR 上で plan を生成し artifact に (ロックがないので refresh しない)
- Merge Queue の並列数を1に制限した上で apply を実行する
- apply に成功したら main ブランチにマージ、そうでなければ plan を再計算して PR に戻す

懸念
- シークレット管理
    - plan のためには state の読み取り権限が必要だが、公開リポジトリでこれは難しい
- 管理コスト
    - それなりに複雑なワークフローが必要になる
changes: ${{ steps.filter.outputs.changes }}
steps:
- uses: actions/checkout@v5.0.0
- uses: dorny/paths-filter@v3.0.2

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'Terraform' step
Uses Step: filter
uses 'dorny/paths-filter' with ref 'v3.0.2', not a pinned commit hash
@tosuke

tosuke commented Oct 27, 2025

Copy link
Copy Markdown
Contributor Author

changes に引っ掛かったら environment を使うようにするという手もある

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants