レガシーな開発現場にUTを入れてCIを導入した話
by
Leko
Link
Embed
Share
Beginning
This slide
Copy link URL
Copy link URL
Copy iframe embed code
Copy iframe embed code
Copy javascript embed code
Copy javascript embed code
Share
Tweet
Share
Tweet
Slide 1
Slide 1 text
レガシーな開発現場にUTを 入れてCIを導入した話 井上 真吾
Slide 2
Slide 2 text
24歳、鎌倉住まい ブログ: http://leko.jp Twitter: @L_e_k_o Github: Leko 会社:Hamee株式会社 QAIT リーダー 入社2年半、小田原、目黒勤務 品質向上と自動化を追い求めるチーム 井上 真吾(れこ)
Slide 3
Slide 3 text
話すこと、話さないこと ● 話します ○ つらみを救ってくれたのは CIでした ● 話せないです ○ イケてる会社の最新の事情 ○ 技術負債と向き合う話
Slide 4
Slide 4 text
2500社近くが導入する、ECのバックエンド一元管理サービス 自社。内製で提供するWebアプリは30近く サービス開始から9年目 PHP, Java, Rubyが主 サポート切れてるバージョンを使ってるサービスも... サービス開始当初は僅かでも経験があれば即採用 仕様書レス、テストレス、レビューレス、仕様レス 動作は手で確かめる。ただしい動きは誰かが知ってる、はず レガシーな開発現場
Slide 5
Slide 5 text
● 一部上場、予算も増えて人も増える ○ 案件開始 → 仕様書もテストもない → 聞いてもわからない → とにかくコード読む ○ 案件ついでにリファクタ → 安定のデグレ → 誰も触れない化物 ○ 仕様書書き出す → 機能が多すぎる → 形骸化or中途半端な情報 ○ レビューなし → しょうもない凡ミス → 何も学ばないで二の鉄を踏む ○ スピード出ない、品質担保で精一杯 ● 契約企業が増えれば負荷も増えるし責任も増す ○ 寿命によるハードウェア障害、スロークエリ、 CPU高沸 etc… ● でも売れてる ○ 売れたまま、スピード感が出せるように加速させたい レガシーが悪いわけではない、が。
Slide 6
Slide 6 text
● 新卒が「技術力足りない」と思うくらいには低かった ● 人も実力者も増え、今やそうではなくなった よろしい、ならば改善だ
Slide 7
Slide 7 text
● コードレビュー開始 ● Doxygen入れてAPIドキュメント生成 ● 仕様置き場の確保、旗振り ● General logをパースしてERD作成 ● UT書き始める。UT書く案件にはベネフィット持たせて書かせる ● ST書き始める。とてもじゃないけど手が回らない ● オフショアに頼る。ベトナムに飛んでコミュニケーション取る ● オフショアといい感じにUT埋めてる ← いまここ 改善の前段
Slide 8
Slide 8 text
要は長い、煩雑 メンバーが増えると こっちの管理コスト 増加、凡ミス頻発。 当初の業務フロー
Slide 9
Slide 9 text
テスト書いてもらうならCIだ
Slide 10
Slide 10 text
● 社内にCIという文化がない ● もう勢いで入れてしまえ、あとでその有用性を分からせる ● 動作環境がめちゃくちゃレガシー ○ CircleCI(phpenv)ではサポートしてくれてないバージョン ○ chefで作られた超大なレシピ ● Dockerで凌ぐ ○ Packer + Dockerでコンテナ生成 ○ 環境がchef製なのが幸いした CIへの壁
Slide 11
Slide 11 text
CI後の業務フロー 手元でテストを実行 する という不確実な手順 を排除
Slide 12
Slide 12 text
● ユニットテストを書く文化づくり ● 効率化、省力化の文化づくり ● 他の製品へのCI横展開 ○ Java製アプリのビルド・デプロイ自動化 ● ドキュメントもCIしたい。lintかけたい。 ● 成果:バグ数軽減、精神衛生向上 ● ゆくゆくは開発効率の向上につなげたい。 今後
Slide 13
Slide 13 text
ありがとうございました! これからも レガシーを駆逐し てゆきます