-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Terraformのファイルを分割する #13
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
早い反応有難うございます!助かります~
分ける部分に関しては問題なくできていそうですが、分け方がちょっと美しくないなぁと思ってしまいます。ちょっと細かいかもしれませんが。
最終的に、今後これを読む人がパッとフォルダを見た瞬間一発で理解できることを目標にしたいので、今できる最大限綺麗にしたいと思っています。
フォルダ分けまでできるのであれば、以下のような形が僕の中では理想ですね
役割をフォルダで分け、その中でリソースの種類をファイルで分けるような形です
- control-plane
- server.tf
- disk.tf
- lb
- server.tf
- disk,tf
- router
- server.tf
- disk.tf
- worker-node
- server.tf
- disk.tf
- network
- external.tf (external switchとsubnet)
- internal.tf (internal switch)
- main.tf
- output.tf (アウトプットはバラバラのフォルダに分けると逆にわかりづらいと思った)
- var.tf
あくまで僕の意見なので、将来これをメンテする人の気持ちを考えて、もし「こっちの方がいいんじゃない?」みたいな案があったら遠慮なく言って下さい!
確認お願いします。 |
分割は完璧です!ありがとう!! あとはチェック通ればマージして大丈夫そう |
Terraform Format and Style
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビュー遅れて申し訳ないです…
完璧ですね!ありがとうございます!
リソース命名とどっちを先に入れるか…って感じですが(片方にはほぼ作り直しをお願いしなければいけないので)
Yuu君はどう考えてます?
お忙しい中すいません。 #17 の進捗的には、あとレビューの修正だけでしょうか? 追記 |
返信遅れてごめんなさい… Terraformだけ変更は流石にやめときたいです |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Asahiくんからしばらく反応がないので、Yuuくんのやつマージしちゃって下さい。
改めてありがとう!引き続きよろしくお願いします
関連
概要
Checklist
備考