プラットフォームリーダーシップレビューガイドライン
目的
このページでは、C1 および C2 変更リクエストを承認する際のプラットフォームリーダーシップレビュアー向けのガイドラインを提供します。目標は、レビューの一貫性を確保し、高重要度変更の品質を向上させることです。
承認できる人物
プラットフォームリーダーシップ承認（platform_leadership_approved ラベル）は、インフラストラクチャ Platforms の IC と EM の両方が提供できます。資格を持つメンバーは以下のとおりです:
- すべてのエンジニアリングマネージャー（EM+）
- Staff+ IC（プリンシパルを含む Staff+ の SRE とバックエンドエンジニア）
レビュアーは変更の著者とは異なるチームである必要があります。これにより、独立した視点が確保され、作業に近い人々が見落とす可能性のある問題を発見するのに役立ちます。
承認できる人物のリストは @gitlab-org/saas-platforms/change-review-leadership グループに記載されています。変更リクエスト Issue でこのグループにメンションして承認を得てください。
レビューガイドライン
C1 または C2 変更リクエストをレビューする際は、以下を確認してください:
1. 本番前の検証
変更が必要な理由の動機が説明文から明確であり、詳細については Issue へのリンクが含まれていること。
同じ変更計画が、同じ手順を使用して非本番環境で実行済みであること。
- 変更がステージングまたはその他の非本番環境でテスト済みであることを確認する
- テスト環境の実行が本番向けに提案された手順と同一であることを確認する
- 非本番テスト中に発見された問題が対処済みであることを確認する
2. ロールバック計画の品質
ロールバック計画が文書化されており、任意の SRE が実行できるほど詳細であること。
- 変更リクエストにロールバック計画が明示的に文書化されていることを確認する
- ロールバック手順が具体的で実行可能であること（「変更を元に戻す」などの漠然とした表現でないこと）を確認する
- 変更に不慣れな SRE が追加のコンテキストなしにロールバックを実行できることを確認する
- ロールバックの時間の見積もりが提供されており、現実的であることを確認する
3. モニタリングと検証
モニタリングリンクが提供されており、モニタリングを確認するための明示的な変更手順と適用可能な場合の検証内容があること。
- 関連するモニタリングダッシュボードのリンクが含まれていることを確認する
- 変更計画に実行中および実行後のモニタリング確認のための明示的な手順が含まれていることを確認する
- 成功基準が定義されていることを確認する（「この変更が成功した」とはどのような状態か？）
- ロールバック計画をトリガーする際の明確な指標があることを確認する
ラベルの適用
変更リクエストが上記のすべての基準を満たしていることを確認した後、Issue に platform_leadership_approved ラベルを適用してください。
変更リクエストが基準を満たしていない場合は、承認が可能になる前に改善が必要な点について具体的なフィードバックを提供してください。
