サーバーの削除・再生成をまとめて実行する方法をPRで対応したい

[yasulab]

現状では次のどちらかで対応する必要があります。

1. 管理者が Slack 上で削除し、Travis をリスタート
2. 運営者が削除するPRを出し、再度作成するPRを出す => うまく動いてなさそう コーダー道場　市川真間の削除依頼です(削除後再依頼して再構築します) #68 (comment)

CoderDojo 運営者から「サーバーを一から作り直したい」という要望がよく来るので、「サーバーを初期状態に戻す」という要望をPRで受け付けられるメソッドがあると良さそうです 🤔💭 cc/ @miya0001

cf. #50

[miya0001]

そうですねー、でもちょっと難しいかもです。
プルリクストの時の差分を取れるといいんですけどね。要はどの行が削除されたかをTravis CI側で知る方法があればいけるのですが、現状はその方法がわかってないので、どのサーバーを削除すべきかを判別する方法がないんです。

[nalabjp]

経緯ややりたいことなど、ちゃんと理解できている自信はないので的外れでしたらゴメンナサイ。

どの行が削除されたかをTravis CI側で知る方法があれば

だいぶ雑な作りなんですが、こういうことだったりしますでしょうか？
nalabjp#1
https://travis-ci.org/nalabjp/dojopaas/builds/290085095#L472-L487

コミットハッシュはTravis CIのビルドごとの環境変数から取得しました。
https://docs.travis-ci.com/user/environment-variables/

[yasulab]

運営者が削除するPRを出し、再度作成するPRを出す

何度か Restart してみましたが、そもそも servers.csv を削除してもサーバーが削除されてなさそうです 🤔💦
#68 (comment)

2. の方法がうまくいかなさそうなので、現状では 1. で対応するしかなさそう 😓

[miya0001]

現時点で自動削除はまったく対応してないので、slackのコマンドで /sacloud destroy をするかさくらのクラウドのコントロールパネルで削除するかしないと削除されないです。
（そもそも頻繁につくりなおすことを想定してないんですよね。汗）

でもこれ実際問題として /sacloud で削除するのめんどくさいですか？
現状であれば以下の手順です。

1. /sacloud haltで停止して/sacloud destroyでサーバーを削除
2. Travis CIをRestartすれば新しいサーバーができあがる。

この間に必要なやりとりはサーバーを削除してくださいという何らかの方法での連絡と終わりましたというやりとりだけです。

プルリクエストのやりとりでサーバーをつくりなおすには最低2回のプルリクエストのやりとりが必要になります。

あっ、要は開発コストの問題というかぶっちゃけそれほど全体の手間がかわらないのなら、頑張って開発するモチベーションが。。。笑
でも他の人もプルリクくれたりしていろいろ改善できるならなんでもやってみたいですね！

@nalabjp

もしかしたらそうかもですね。コミットのハッシュが取れるならdiffをパースして削除されたサーバーのIDをとれるかもですね。

[yasulab]

現時点で自動削除はまったく対応してない

おぉ、勘違いしてました 💦

でもこれ実際問題として /sacloud で削除するのめんどくさいですか？

この作業に到るまでが負担がかかっていて辛いですね 😓 問い合わせが来る度に #49 のような対応を僕がしているのですが、このままだとマズいなという危機感を持っています💦

技術的に解決する前に、運用で解決するという手もありそうですね。例えば「初期化したい場合」「SSHで接続できなかった場合」みたいなFAQを README に書くだけでも違うのかも 🤔💭 (試してみる価値はありそう)

cf. #57

[miya0001]

なるほど。オペレーションミスのサポートとか入り出すと確かに大変ですね。。。

[yasulab]

例えば「初期化したい場合」「SSHで接続できなかった場合」みたいなFAQを README に書くだけでも違うのかも 🤔💭 (試してみる価値はありそう)

まずはこちらで対応しました ;) README での書き足しでうまくいくか様子を見つつ、それでもダメであれば技術的な解決策を検討しようと思います🤔💭

[yasulab]

あとすみません、READMEやドキュメントを直す程度のコミットに毎回PR出すのが面倒なので Include administrators の項目のチェックを外させてもらいました 🔓 💦
https://github.com/coderdojo-japan/dojopaas/settings/branches/master