Skip to content

ci(prod-deploy): 本番にもGo関数デプロイ工程を追加(dev同等・本番設定)#134

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/prod-deploy-go-parity-5520
Jul 1, 2026
Merged

ci(prod-deploy): 本番にもGo関数デプロイ工程を追加(dev同等・本番設定)#134
cursor[bot] merged 1 commit into
mainfrom
cursor/prod-deploy-go-parity-5520

Conversation

@kojira

@kojira kojira commented Jul 1, 2026

Copy link
Copy Markdown
Member

背景(重大な欠陥の修正)

prod-deploy.ymlfirebase deploy(フロント+Node関数)のみで、Go関数(userOGPGo/statusGo 等)を一切デプロイしていなかった。このままだと env/dev → env/prod#130)を反映しても、Go移植は本番に反映されない(プロモーション経路が未完成)。dev にだけ Go デプロイを入れて本番へ入れ忘れていた私のミス。

変更

dev-deploy.yml と同等の工程を prod-deploy.yml に本番設定で移植:

  • go build/go vet + Firestoreエミュレータ統合テスト(デプロイ前ゲート)
  • OGPサンプルQCアーティファクト生成(ogp-sample
  • GCPサービス有効化 / Pub/Subトピック作成 / OAuth・base_url 取得 / App Engineロケーション解決
  • 全Go関数(HTTP 6本+スケジュール4本)の並列デプロイ + Cloud Scheduler ジョブ作成/更新

本番固有値(dev との差分)

項目 dev prod
project d-shrine-dev d-shrine
STORAGE_BUCKET_NAME d-shrine-dev.appspot.com d-shrine.appspot.com
SANPAI_NEXT_TIME_SECONDS 60 300
OGP_PROJECT_ID d-shrine-dev d-shrine
認証 DEV_GCLOUD_SERVICE_KEY PROD_GCLOUD_SERVICE_KEY

Node版の buggetName = ${projectID}.appspot.com / next_time = (projectID=='d-shrine')?300:60 と一致。

検証

  • actionlint .github/workflows/prod-deploy.yml(および dev): パス。

プロモーション順序(重要)

この修正が env/prod に入った状態で env/dev → env/prod を反映して初めて Go関数が本番デプロイされる。したがって: 本PR → mainenv/devenv/prod(#130) の順で反映する。

Open in Web Open in Cursor 

prod-deploy.ymlはfirebase deploy(フロント+Node関数)のみで、Go関数(userOGPGo等)を
一切デプロイしていなかった。これではenv/prodへ反映してもGo移植が本番に出ない。
dev-deploy.ymlと同等の工程を本番設定で移植する:
- go build/vet + Firestoreエミュレータ統合テスト(デプロイ前ゲート)
- OGPサンプルQCアーティファクト生成
- GCPサービス有効化・Pub/Subトピック作成・OAuth/base_url取得・App Engineロケーション解決
- 全Go関数(HTTP6+スケジュール4)の並列デプロイ + Cloud Schedulerジョブ作成/更新
本番固有値: project=d-shrine, bucket=d-shrine.appspot.com,
SANPAI_NEXT_TIME_SECONDS=300, OGP_PROJECT_ID=d-shrine
(Node版 buggetName=projectID+.appspot.com / next_time=300 と一致)

Co-authored-by: kojira <kojira@users.noreply.github.com>
@cursor cursor Bot merged commit 841f5cf into main Jul 1, 2026
3 checks passed
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.

2 participants