Skip to content

Latest commit

 

History

History
127 lines (91 loc) · 4.31 KB

File metadata and controls

127 lines (91 loc) · 4.31 KB

概要

  • なぜこの変更をするのか、
  • 課題は何か、
  • これによってどう解決されるのか、
  • など、この変更に対する概要を記載

環境変数(SSMパラメータ)

  • 環境変数やSSMパラメータに変更を加えたか否か
    • 変更したとしたらどのような変更か
  • 変更した場合は以下の作業を行う
    • ssmリポジトリに該当の値を追加し本PRと同時にPRを出す
    • ステージング環境のSSMへ該当のパラメータを追加
    • Slackでエンジニア全員へこのPRの内容を共有
    • 必要であれば.envrc.sample へ環境変数名とその用途の詳細な説明を追加
    • 必要であればmasterへマージされた時に走るCI環境への追加
      • 必要であればCircleCI Contextsにも追加

関連URL

影響範囲(ユーザ)

  • 影響を与えるユーザは誰か

影響範囲(システム)

  • 影響を与えるシステムはどこか
  • サーバレス
  • フロントエンド
  • プライベートチェーン

影響範囲(開発者)

  • 開発チームに共有すべきことはあるか

技術的変更点概要

  • なにをどう変更したか
  • ロジックがどういう手順で動くのか、
  • DBからどういうクエリで何をとってそれに何を処理するのか、

使い方

  • 使い方の説明
  • バグの場合は再現条件

DBやDBへのクエリに対する変更

  • DBのスキーマに変更があるか
  • 変更されるクエリはあるか
    • あるならどのような変更か
      • 変更前:
      • 変更後:
  • 新規に追加されるクエリ
  • 観点:
    • index周りの指定は適切か
    • 想定される対象のデータ量
    • 想定されるデータ量が大量のとき大丈夫か (ダメな場合、いつまでにどうにかする予定があるのか、忘れないために別チケットを用意したか)

ブロックチェーンへの影響

  • ブロックチェーンを参照するか
  • ブロックチェーンへのトランザクションが発生するか

トークンの扱いに関する修正か

  • Yes
    • 投げ銭におけるワンタイムトークンなど、フロントエンドに対してフェイルセーフな設計となっているか
  • No

CDN(Cloudfront)への修正があるか

  • 無い
  • ある
    • あるとしたらその修正内容をConfluenceにまとめたか

ElasticSearchへの修正があるか

  • 無い
  • ある
    • あるとしたらその修正内容はコード化したか

CloudFormationスタック間の依存関係に変更はあるか

  • 無い
  • ある
    • あるとしたらその修正をCIへ反映したか

個人情報の取り扱いに変更のあるリリースか

  • メールアドレスや住所情報等、個人情報にあたるものは細心の注意が必要

ロギング

  • CloudWatch等、AWS標準以外のログを考慮する必要があるか
    • あるとしたらいつ、どこで、なぜ、どこに、どのようなログが吐かれるか

アラーム

  • アラームが必要か
    • 必要
    • 不要
  • 必要な場合はアラームをコードに追加したか
    • した
    • していない

ユニットテスト

  • ユニットテストが書かれているか
  • 書かれていない場合、妥当な理由があるか。それは何か

テスト結果とテスト項目

  • テストする際の項目を、このように、チェック可能な形式で記載する。
  • テストしたらチェックを入れていく。

保留した項目とTODOリスト

箇条書きで書く。可能な限り次のチケットを作る。

注意点・その他

  • この作業で特に注意する点があれば記載する
  • その他、補足事項があれば記載する

レビュワーに依頼したいこと

  • 内容を確認し、問題なければマージしてください。