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

change to GitHub Flow #370

Merged
merged 7 commits into from Aug 2, 2019
Merged

change to GitHub Flow #370

merged 7 commits into from Aug 2, 2019

Conversation

sot528
Copy link
Contributor

@sot528 sot528 commented Aug 2, 2019

概要

  • なぜこの変更をするのか、
    • serverless-applicationをGitHub Flowへ変更
  • 課題は何か、
    • 工数削減

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

  • 環境変数やSSMパラメータに変更を加えたか否か
    • No

影響範囲(システム)

  • 影響を与えるシステムはどこか
  • サーバレス

影響範囲(開発者)

  • 開発チームに共有すべきことはあるか
    • ブランチ戦略の変更
    • 本PRがマージされたらマージした人が通知

技術的変更点概要

  • なにをどう変更したか
    • GitHubのデフォルトブランチをmasterに
    • CI構成を変更

CIの変更点

以下のようなCIワークフローとした。

フォークしたリポジトリ

スクリーンショット 2019-08-02 12 17 53

masterにマージしたときのステージング

スクリーンショット 2019-08-02 12 55 45

masterにマージしたときの本番環境

スクリーンショット 2019-08-02 12 55 52

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

  • なし

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

  • なし

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

  • No

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

  • 無い

ElasticSearchへの修正があるか

  • 無い

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

  • 無い

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

  • なし

テスト結果とテスト項目

  • 石井環境で問題なく動作することを確認
    • 3つのワークフローともに問題なし

注意点・その他

  • ついでにalarmsのデプロイ処理も追加しました。

お願いしたいこと

PRをマージ後、そのままdevelopをmasterにマージしてステージング環境へ反映されることをご確認ください。本番環境へのビルドは放置でおkです。

@sot528 sot528 marked this pull request as ready for review August 2, 2019 04:19
executor:
name: python3_6_1

executor: python3_6_1
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

リファクタリングして省略記法へ変更。

steps:
- deploy-resources:
target: 'function'

deploy-function-production: *deploy-function
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ワークフローを分けたためproduction専用のジョブは不要となりました。

@@ -303,7 +286,7 @@ jobs:
- v1-npm-dependencies-

- run:
command: npm i
command: npm ci
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ついでにpackage-lock.jsonを使用するよう変更。

@sot528 sot528 requested a review from a team August 2, 2019 04:32
@ghost ghost requested review from keillera and removed request for a team August 2, 2019 04:32
@keillera
Copy link
Member

keillera commented Aug 2, 2019

LGTM です!

@keillera keillera merged commit 07ad9b6 into AlisProject:develop Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants