CDワークフローを構築#19
Merged
Merged
Conversation
masaya-osuga
requested review from
a team,
KAZZ00777,
hikaru-0602,
kantacky and
zakiPoteto
Dockerfile のビルド対象と deploy.yml の Cloud Run Service 更新対象に announcement-api / user-api を追加し、3 つの HTTP API すべてが イメージに同梱・デプロイされるようにした。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
並行グループ名のプレフィックス ci- を外し、ワークフロー名と ref のみで一意になるよう簡素化した。あわせて mise の version ピン留めを廃止し、デフォルトの最新版を使うようにした。
ファイル名とワークフロー名を CD に統一し、手動実行の入力を environment から environment_name (必須・デフォルトなし) に変更した。 並行グループ名も環境ごとに分離する形へ見直し、誤った環境への 取り違えを防ぐようにした。ビルド・デプロイ処理自体は従来と同等。
単一サービス前提だった出力を見直し、cmd/* から デプロイ matrix を自動生成する set-matrix ステップを追加した。 - service_name 出力を廃止し、サービス/ジョブ名へ付与する service_suffix 出力に置き換え（prod は空） - image_tag をサービス名非依存にし、全 cmd ターゲットで 1 つのイメージを共有する形へ変更 - cmd/* を走査し、末尾 "-api" を Cloud Run Service、 それ以外を Job として分類した matrix を出力 このコミットでは set-env アクション単体の出力定義のみを変更し、 利用側ワークフローの変更は次コミットで扱う。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
単一サービスのデプロイから、cmd/* 由来の matrix で各ターゲットを 並列デプロイする方式へ切り替えた。サービス/ジョブの増減に ワークフロー編集なしで自動追従できる。 - build-and-push を 1 回だけ実行し、共有イメージを生成・プッシュ - deploy ジョブに set-env が出力する matrix を適用し、 service_suffix と組み合わせてサービス/ジョブ名を構築 - *-api は deploy-cloudrun で Service デプロイ、それ以外は gcloud run jobs deploy で Job を作成/更新（実行はしない）。 いずれも共有イメージの起動コマンドを /bin/<name> に切り替え - 不要になった env（REGION/AR_REPOSITORY/IMAGE_NAME）を削除 - ポリシーに従い各 action をタグから SHA 固定に変更 （Renovate 用にタグをコメント併記） Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
レビュー全体的に CD ワークフローの初期構築として良い方向です。いくつか気になる点を挙げます。 1.
|
CD ワークフローを cmd/* 由来の matrix による並列デプロイへ刷新
kantacky
reviewed
Jun 20, 2026
kantacky
reviewed
Jun 20, 2026
kantacky
reviewed
Jun 20, 2026
kantacky
reviewed
Jun 20, 2026
workflow_dispatch の選択肢 prd と判定条件 prod の不一致を解消し、 prd 選択時にサフィックスが正しく空になるよう修正。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
サブモジュールは存在せず、Docker ビルドは git 履歴も使わないため submodules: true と fetch-depth: 0 を削除。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
kantacky
approved these changes
Jun 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.