fork先からのPRだとterraform planが走らない

[sksat]

• Add my GitHub id #32

非常に正しい挙動だが，今回は困る

[sksat]

無の認証情報を撒いてterraform validateまではGitHub Actionsでやるのはアリかもしれん

[sksat]

https://www.terraform.io/cli/commands/validate

The terraform validate command validates the configuration files in a directory, referring only to the configuration and not accessing any remote services such as remote state, provider APIs, etc.

そうよね

[sksat]

が，しかしその場合でもterraform planはしたいので，一旦別ブランチにマージして様子を見る，みたいなかんじになるなあ(main自動applyはやりたいので)

[sksat]

これはできるが，1人増える度に僕がマージボタン2回押さなきゃいけないのはちょっと嫌だな

[sksat]

どうにかしてTerraform Cloud氏が毎回approve and runみたいなやつ生やしてくれればいいんだけどなあ

[sksat]

2段階マージはあくまで許容するものとして，僕のポチポチの手間を減らす方向で行くのはアリかもしれない

[sksat]

とりあえずterraform validateは入れられるんで入れるか

[sksat]

入れた

• terraform validate on GitHub Actions #36

[sksat]

mainのon: pushで確実にmainにrebaseされたブランチを切るなどはあるな

[sksat]

GitHub Actionsでやるのはダルい上にログが出て渋いなどがあるので，planだけ発射して実行結果のリンクがPR commentで生えればいい説もある

[sksat]

on: pull_request_targetにした上でlabelを付けたら走るようにしてAPI経由でplan，かなあ

[sksat]

よく考えたらVCS-driven workflowだからterraform loginしてしまってplanするのでそんなに問題無いな

[sksat]

これでよさそう

• fork先からのPR用にGitHub Actions経由でterraform planする #41

[sksat]

ミスってたんで直した

• terraform planのCIの修正 #42

[sksat]

いいかんじになったので閉じ

• Add my GitHub id #32 (comment)