r/programming🔥 48
💬 20

GitHub FlowとTrunk-based開発、どっちがボトルネックになる?インタラクティブなシミュレーターで可視化しよう

ny3000
約22時間前

ディスカッション (21件)

0
ny3000OP
👍48約22時間前

GitHub FlowとTrunk-based開発(トランクベース開発)のどちらを採用すべきか迷っていませんか?このシミュレーターを使えば、それぞれのワークフローでどこにタスクの溜まり(キュー)が発生しやすいのか、一目で把握できます。チームの生産性を最大化するためのヒントとして、ぜひ触ってみてください。

1
gredr👍 75
約22時間前

最初に結論ありきで、それを正当化するためにシミュレーションを作ったって感じ?

2
ny3000
👍2約21時間前

はは、その可能性もあるね!ただ個人的には、Thoughtworksで実際にやっていた頃(CDが発明された場所でもあるし)に、15年間現場でチームがこれらを実践するのを見てきた経験に基づいているよ。

3
paul_h
👍7約21時間前

いい仕事だね。ちなみにThoughtworksには何年いたの?

4
ny3000
👍3約21時間前

確か2013年〜2019年と、2022年〜2024年だったと思う

5
gredr
👍6約21時間前

自分は短命ブランチとPRを使った「GitHub flow」が経験上しっくりくるかな。Trunk-based developmentもかなり良いんだけど、なぜか使わなくなったんだよね…理由はよく分からないけど。フィーチャーフラグは素晴らしいけど、リファクタリング全体をフラグ化するのは難しいし、複数のAPIバージョンを管理するのは大変だしね。

GitFlowは…個人的にはやりすぎだと思う。誰が何のためにあんな複雑なことをしたがるのか分からない。カーネル開発みたいに、世界中の何千人もの人から大量のパッチを受け取って、いくつものバージョンを並行管理するなら必要かもしれないけど、自分たちのような一般的な開発には当てはまらない気がするな。

6
ny3000
👍3約21時間前

リファクタリング全体をフィーチャーフラグにするのは無理

データベースが絡んでくると特に複雑になるよね。でも、ブランチを使うよりはマシだと思うな。あと、リファクタリングの影響範囲が広いなら「Branch by abstraction」や「Dark launching」の方が安全かもしれない。最近フィーチャーフラグ周辺のビジネスが流行ってるけど、何でもフラグを使えばいいってわけじゃないしね。

7
ChadtheWad
👍4約19時間前

機能フラグは素晴らしいけど、リファクタリング全体を機能フラグで制御するのは無理だし、複数のAPIバージョンを維持するのは面倒だよね。

実はそのケースで機能フラグに助けられた経験があるんだ。重要なワークフローソリューションを新システムへ移行する際に約3000行のコード変更が必要だったんだけど、結局それを機能フラグで隠したんだ。良かった点は、エンドツーエンドの整合性テストを組み込んで、新旧両方のロジックが同じ挙動をするか確認したことだね。おかげで設計中にレガシー側に何か変更が入っても、テストで検知して適切に更新できたんだ。

8
bunoso
👍5約22時間前

アニメーション最高だし、ログのメッセージも笑える!バックログのタスクやバグが山積みになってて、マネージャーに「これにAIをどう使う?」って言われても結局作業が進まず、キューがどんどん溜まっていくっていう…。なんでだろうね。全体的にすごく面白い!スマホだと縦長レイアウトの方が良さそうだから、後でノートPCでチェックしてみるよ

9
SeniorIdiot
👍8約21時間前
  1. レビュー、待ち、再コーディング、マージを動かすパラメータはどうなってるの?
  2. 「LGTM」レビューも必要じゃない?成功する時もあれば、後からバグや手戻りが自動的に発生するようなやつ。
  3. あと、マージ/ビルドしかないのに「CI」っていうステップがあるのが謎。

追伸:FB/PRワークフローには断固反対だけど、無限タスクをシミュレートするんじゃなくて、もっと正直にやってほしいかな。

10
ny3000
👍1約21時間前
  1. いい指摘だね。シミュレーションには20個くらいのコンフィグ用ノブがあるんだけど、今はユーザーからは見えないようになってるんだ。
  2. 適当なLGTMですでに本番バグは発生するようにしてるけど、ログだと少しわかりにくいよね。
  3. それも同意。Git flowには「Build」があるけど、GitHub flowはたまに試してくれるから少し甘く見てたかも。

V2で改善しておくよ…あ、MVP v2ね。

11
redbo
👍20約21時間前

Git-flowの魅力が全く理解できない。

12
civildisobedient
👍6約18時間前

長期間ブランチを残す手法には向いているね。でも、そもそも「なんでそんなにブランチが長期間生きているの?」という疑問が当然湧いてくる。それは大抵、組織の機能不全やアジリティの欠如を反映しているものだから、それが存在すること自体を否定はしないけど、それを「推奨」する奴がいたら正気を疑うね。

13
Solonotix
👍12約20時間前

他のプロセスに対してかなり悲観的だったね。特に、組み込まれているバグ発生率がTrunk-Basedに有利なように歪められているように感じる。それに「待機中」という極めて悲観的なステータスがあって、他のすべてのステータスの合計よりも多くのアイテムがそこに溜まってしまったんだ。

だから、あるアプローチが「バグが少なくて高速」と定義されていれば、結果もそうなってしまうのは当然だよね。もしすべての手法でバグ発生率を共通にすれば、スタートラインとして公平だろう。4人の開発者がペアの2人よりもアウトプットが低いと決めつけるのは非常にバイアスがかかっている。だって、実際に同時進行している作業の数が少ないんだからさ。

追記:GitHub Flowで「リリース」ボタンを押した際、2回もPre-Prod(本番前環境)にステージングされていたアイテムがすべて最初からやり直しになったんだ。これって一体どういうクソ仕様なの!?その結果は除外したけどね。

それぞれ1000時間分回してみた結果がこれだ。

名前機能バグWIPキュー
GitHub Flow (#1)17152358
GitHub Flow (#3)13141953
GitHub Flow w/ QA1542544
Git Flow112454
Trunk-Based671215

GitHub Flowをやり直したのは、1回目で数値を記録中に「一時停止」をしなかったから。2回目は最大速度で実行したせいか変な挙動をしたから破棄した。QA付きの実行は、それが結果に奇妙な偏りを与えていないか確認するために追加したんだ。

結論として、君のシミュレーションは極めてバイアスがかかっていると言わざるを得ないね。

14
ny3000
👍3約20時間前

サンクス。フィードバックを元に透明性を上げたから、バグが発生した時に何が原因でそうなったのかログに表示されるようになったよ。

バグ発生の計算式は全モード共通だけど、TBDは構造的にバッチサイズが1になるからバグ率が低くなるんだ。シミュレーション自体は、なぜ数値が乖離するのかを説明するものではないからね。

確かにTBDに対しては非常に偏っているよ。実プロジェクトでTBDと他のモードの両方を経験して、良さを知っているからね。これはあくまで裏紙に書いたようなお遊びのシミュレーションであって、学術論文じゃないんだ。DORAの連中がすでにそういうのはやってるしね。

15
r2p42
👍5約20時間前

誰かこの結論を噛み砕いてくれない?ドットが跳ね回ってるのを見たんだけど、これって良いの?悪いの?

16
audentis
👍15約20時間前

要するにモデルが間違っているから、そこから結論を導き出すことはできないってことだよ。

17
ny3000
約20時間前

まず、これはお遊びレベルのシミュレーションであって研究論文じゃないってことを理解してくれ。各ドットは作業項目で、青はストーリー、オレンジはレビューや手戻りだ。GitHub Flowの「待機中」カラムが埋まっていくのを見てほしい。それが作業の停滞だ。TBDタブに切り替えると、パイプラインは空のままだろう。何も待機していないからね。一番下の数値は、完了、WIP、キュー、リードタイムを表しているよ。

18
mothzilla
👍2約19時間前

次は顧客へのFTP直アップロードでやってみてくれ。

19
baseballlover723
👍2約18時間前

いいUIだけど、ひどくバイアスのかかったモデルだね。

20
Agent7619
👍1約18時間前

色使いに少しはコントラストをつけてくれよ。文字が全く読めないんだが。