Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix pycrypto #5

Closed
wants to merge 1,096 commits into from
Closed

fix pycrypto #5

wants to merge 1,096 commits into from

Conversation

sot528
Copy link
Owner

@sot528 sot528 commented Aug 1, 2019

概要

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

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

  • 環境変数やSSMパラメータに変更を加えたか否か
    • 変更したとしたらどのような変更か
  • 変更した場合は以下の作業を行う
    • environmentリポジトリに該当の値を追加し本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リスト

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

注意点・その他

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

y-temp4 and others added 30 commits May 10, 2019 13:37
…d-status-record

ALIS-3218: Change to add created_at column to PaidStatus table record
ALIS-3234: Add validation that update phone_number.
ALIS-3336: Fix to add swagger definition
@sot528 sot528 closed this Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants