今からでも間に合うphp5.5からphp7.2にバージョンアップした話 #phpcon

今からでも間に合う php5.5からphp7.2にバージョンアップした話菅原佑太 / kosa3 株式会社ウエディングパーク

・結婚・婚約指輪の口コミサイト　「Ringraph」の運用・開発責任者・サーバーサイドエンジニア・Serverless Framework, Nuxt.js勉強中菅原佑太株式会社ウエディングパーク新卒3年目/チーフエンジニア
kosa3

・ウエディングパークの姉妹サイト・2015年 9月からサービススタート(3歳) ・クチコミ数2万件以上・php + CodeIgniterで構成

それは突然のことでした…

「社内で実施しているセキュリティチェックで１つお願いしたいことがあるんだが...」上司

「なんですか？」ぼく

「PHPのバージョンアップをお願いしてもよいかな？」上司

「いつまでにですか？」ぼく

「3ヶ月くらいで…」上司

「ちょ、調査します！！！」ぼく

既に2016年でサポート終了

今回あげるバージョンは7.2

早速、工数見積もり調査を開始

phpの公式ドキュメントから移行について洗い出し、該当するか調べる調査その1

大体つまずきそうなところは分かった！

Docker環境を作り、 phpのバージョンをあげて検証してみる調査その2

$ docker-compose up

まぁまぁここまでは想定内… きっと他のミドルやモジュールの関係だろう

調査結果

・Codeigniter Framework ・CentOS ・Apache ・node.js ・ruby ・composer ・mod_small_light module ・mcrypt
module ・その他モジュールやクライアントライブラリなど PHP以外に必要なバージョンアップ

３ヶ月で終わるのこれ…

急遽エンジニア2人増員

・動くようにする・ログなどのエラーが出ないようにする・リファクタリング(コードレビュー負荷軽減、　余計なロジック周りのテストの影響範囲を抑える) やるべきことやらないこと

バージョンアップPJでの課題3選・暗号化モジュールmcryptの完全非推奨・画像リサイズmod_small_lightのアーキテクチャ変更・既存システムの不具合

・php7.2からmcrypt暗号化モジュールが完全非推奨　=> 全てOpenSSLに変更 mcryptの完全非推奨

mcryptよりOpenSSLの方が処理速度も良いらしい【参照】 http://yut.hatenablog.com/entry/20140228/1393543543 mcryptの完全非推奨

・mod_dimsなど他モジュールで代用・コンテンツサーバーを用意・serverless でリサイズ関数を作成する画像リサイズmod_small_lightのアーキテクチャ変更 mod_small_lightの後継者探し (サーバー負荷も抑えたい)

・mod_dimsなど他モジュールで代用・コンテンツサーバーを用意・serverless でリサイズ関数を作成する画像リサイズmod_small_lightのアーキテクチャ変更 Apache2.4でmod_small_lightの後継者探し (ついでにサーバー負荷も抑えたい) =>姉妹サイトの「DRESPIC」がlambdaでの　画像リサイズを導入していたので横展させてもらう

アーキテクチャ

・全体に及ぶ影響から全範囲を開発・テストする・意図せぬ不具合を発覚（Sentry, 監査）・修正 & 報告書 => 工数に入れていない&対応に時間がかかる既存システムの不具合

まとめ

よかったこと・PJ開発ルールの統一・ミドルのバージョンをモダンにできた・WAF、CloudFrontなどアーキテクチャを追加・SEO対策、パフォーマンス施策など打てた・php-cs-fixerを導入し、コードの統一した・テストコードを導入し、CircleCIで自動テストを運用できた

課題だったこと・全体工数見積もりの算出・既存システムの不具合にかかるリソースコスト・現行システムの運用案件の並行開発

最後に

タイトなスケジュールでも事前にやることやらないことを明確化することで最低限のリソースで進めることができました (今回だけかも…)

でもやっぱりバージョンはこまめにあげましょう

ご静聴ありがとうございました

今からでも間に合うphp5.5からphp7.2にバージョンアップした話 #phpcon

今からでも間に合うphp5.5からphp7.2にバージョンアップした話 #phpcon

More Decks by kosa3

Other Decks in Technology

Featured

Transcript