Merge pull request #327 from AlisProject/release/0.37.0

Release/0.37.0

.circleci/config.yml

@@ -78,6 +78,7 @@ jobs:
             if [ $ALIS_APP_ID ]; then
               . venv/bin/activate
               ./deploy_api.sh
+              ./fix_api.sh
             fi
 
       - store_artifacts:

.envrc.sample

@@ -10,6 +10,7 @@ export ALIS_APP_ID=foobar
 # TODO: SAMがIAM認証をサポートしたらteplate.yamlに集約してこの値とスクリプトを除却する
 #   - see: https://github.com/awslabs/serverless-application-model/issues/248
 export SERVERLESS_REST_API_ID=pe6odilrre
+export SERVERLESS_REST_API_WITH_OAUTH_ID=xxxxxxxxxx
 
 # --- AWS ---
 

.github/pull_request_template.md

@@ -28,6 +28,12 @@
 - フロントエンド
 - プライベートチェーン
 
+## 影響範囲(開発者)
+* 開発チームに共有すべきことはあるか
+  * 環境構築手順に変更はあるか。
+    * ある場合は手順を更新: http://bit.ly/30bfDZF
+
+
 ## 技術的変更点概要
 * なにをどう変更したか
 * ロジックがどういう手順で動くのか、
@@ -62,6 +68,14 @@
 * ロギングは考慮されているか
 * ログをはくとしたらいつ、どこで、どこに、どのようなログが吐かれるか
 
+### アラーム
+* アラームが必要か
+  * 必要
+  * 不要
+* 必要な場合はアラームをコードに追加したか
+  * した
+  * していない
+
 ## ユニットテスト
 * ユニットテストが書かれているか
 * 書かれていない場合、妥当な理由があるか。それは何か

.gitignore

@@ -1,4 +1,5 @@
 *packaged-template.yaml
+*package-template.yaml
 venv
 deploy
 vendor-package
@@ -24,3 +25,6 @@ dynamodb_local*
 shared-local-instance.db
 LICENSE.txt
 README.txt
+
+# etc
+node_modules

README.md

@@ -114,15 +114,15 @@ Specify generated Cognito User Pool ARN to SSM.
 
 
 ### Lambda & API Gateway
-You have to add SNS authentication params to SMS.
+You have to add SNS authentication params to SSM.
 - See: https://github.com/AlisProject/environment
 
 
 ```bash
 ./deploy.sh function && ./deploy.sh api
 ```
 
-You have to add `RestApiArn`, `ApiLambdaRole` and `ElasticSearchEndpoint` to SMS.
+You have to add `RestApiArn`, `ApiLambdaRole` and `ElasticSearchEndpoint` to SSM.
 - See: https://github.com/AlisProject/environment
   - You can use `api-stack-replacer.sh`
 
@@ -158,6 +158,15 @@ aws apigateway  get-rest-apis | jq '.items[] | if .name == "'${ALIS_APP_ID}'api"
 # Set SERVERLESS_REST_API_ID to .envrc
 direnv edit
 
+# Show generated Rest API with OAuth ID
+aws apigateway  get-rest-apis | jq '.items[] | if .name == "'${ALIS_APP_ID}'api-with-oauth" then .id else empty end'
+
+# Set SERVERLESS_REST_API_WITH_OAUTH_ID to .envrc
+direnv edit
+
+## Load envs
+direnv allow
+
 ./fix_api.sh
 ```
 
@@ -186,3 +195,10 @@ Following example is that `ArticlesRecent` function is deployed.
 ```bash
 python make_deploy_zip.py && ./deploy_api_function.py ArticlesRecent
 ```
+
+### CloudWatch Alarm
+For production and staging, you should enable alarms.
+
+```bash
+./deploy.sh apialarms
+```