待って！そのデプロイ本当に大丈夫？✋ - きりだるまの記録帳

全国100万人のエンジニアの皆さんこんにちは、デプロイしてますか？ この記事は、本番環境でやらかしちゃった人 Advent Calendar 2019の5日目の記事です 予想以上にこのアドベントカレンダーが注目されてて、書きながら「これ炎上しないよね…」「なんかマズイこと書いてないよな…」と胃が痛くなってます 自分みたいなザコメンタルにはつらい…めっちゃやむ… この記事はただの不注意が原因のやらかし話です 過度な期待はしないで下さい 追記 さすが3000人以上に購読されてるアドベントカレンダー、アクセス数がすごいことになってますね…怖い… いくつかコメントや言及を頂いたので、追記というのをしておきます テスト環境ないの？ あります タスクAに関してはテスト環境での動作チェックが不十分だった タスクCに関しては焦ってて動作確認せずに出しちゃった(「まぁこれくらいの変更なら大丈夫でしょ」という

[Falky]

レビュー通ってないのにセルフマージできるからよくない。人間の注意力に頼るのは無意味。masterをprotected branchにしましょう。（昨日と同じ話だ…protected branchってみんな知らないものなんだろうか）

[aikawame]

王道あるある話が来た。人間はマルチタスクに向いてない、本当に向いてない…。

[Ikalga]

再発防止策が結局「頑張って気をつけます」になってるのがモヤモヤする。「レビューOKされないとマージできない仕組みにする」「実装者、レビュアーと別のリリース承認者を設けるフローにする」あたりをやりたい

[send]

皆さん、masterブランチはprotectしましょうね！(このアドベントカレンダーで何回これが原因のやつ出てくるんだろ)

[eichisanden]

レビュアーがレビューした流れでマージボタン押すルールにすればと思った

[dagama]

まともなテストをせずに本番リリースしてるのがすべての元凶

[tito1201]

"フルリモートなので周りにヘルプを出せないのもつらいですね" ここも問題では？一人で(おまけにパラレルで)やってると視野狭窄になってミスしがちなので、オブザーバ的役割の人がついてるだけでだいぶ違う気がする。

[bps_tomoya]

"GitLab、調べたらapproveもらわないとmergeできない機能は有料版しかない" 知らなかった。金で殴れるのが一番いいけれど、金で殴れない様々な如何ともしがたい背景が各々にあることも忘れないでおきたい

[ytRino]

merge即deployはカナリアとか周辺含めて整備しないと怖いなあ /"落ちたりするとTwitterで騒がれるくらいには大きいサービス"やってる企業なら有料版を使わない合理的な理由はないよなあ

[rryu]

本番作業のながら作業はもう確実に死亡フラグ。ミス＝死みたいな作業をやるにはちゃんとした覚悟が必要になる。

[hylom]

こういうトラブルは良くあるようで、そのためレビューアーがOKを出さないとマージを実行できないような仕組みになっている開発支援ツールがある

[osnq]

恐ろしいの見すぎてて大したことないって思っちゃうなｗ怖い怖い。

[mobits]

説明下手すぎない？

[kamei_rio]

"作業する際は、関係ないウィンドウを閉じよう" 増えすぎたら危険なのは確かに

[jacoby]

並行作業やり過ぎに注意。

[new3]

レビュー制度あるのにレビュー通っていないのマージできる仕組みが良くない。しかしそれが有料で課金殴りできる資金がないとなるとなぁ…

[wordi]

細かい事だげと、自分はタスク別にwipブランチ作って作業する時はOSの仮想デスクトップ機能使って画面を切り替えてる

[zuzu_sion]

あるあるｗ　ただフルリモートでもヘルプ出せる環境の用意がないのは可哀そう。。上司と通話とかできない感じかな？

[delphinus35]

読んでいて「フフッ」って半笑いになる。焦ってマージするところとか。あるよねー

[tyoro1210]

構造的な問題ではあるが、それをリモートでやってるバイトに改善しろってのも酷な話しである(提案くらいはよさそう)。 バイトとしては 「頑張る」しかないし、むしろ会社側がちゃんと整備してやってくれ。

[masaru_b_cl]

人は間違うので、ぜひシステムで解決して欲しい(し、そこにお金掛けられる組織であって欲しい) > “レビュー通ってないpull reqはマージできないように”

[koogawa]

本番環境でやらかしちゃった人 Advent Calendar 2019の5日目の記事！

[daira4000]

焦って対応間違えるのはあるある。ミスる要因はシステム的にできなくしようね。

[kubecorn]

“レビュー通ってないpull reqはマージできないようにしないの？ これが本質な気がしてきた” コメントからさっそく改善につながっててよき

[z1h4784]

本番環境に自動デプロイは怖くて導入できない。ワンステップ置きたい

[yhachisu]

追記読んだ。gitlab無料版使ってる会社が悪い！この記事とブコメをエラい人に見せて有料版にさせるんだ！同僚みんなに感謝されるぞ！

[shoh8]

“これ以降、デプロイ前後に小休止を入れるようにした”焦りは何も生み出さないから、私もこれは肝に銘じている。あとはなるべく一人作業にならないような開発環境作り、レビューとマージフロー

[YukeSkywalker]

エディタとかターミナルのシングルトンは大事。

[inductor]

クロスレビュー大事

[razokulover]

自分はマルチタスク得意なので似たようなことをよくやってるんだけどまだ事故ってないだけかもしれん。気をつけよ

[ksugimori]

たしかにメーデー感ある。メーデー的にはデプロイ前チェックリストが必要かしら。