DB削除前にCloud Runサービスを停止して接続を切断

[komagata]

Summary

• Cloud Runサービスを--max-instances=0で停止することで、DBへの接続を確実に切断
• Cloud SQLでは管理者権限がないためpg_terminate_backendが使えない問題を解決
• 30秒の待機時間で接続のドレインを確保

変更内容

1. StopCloudRunステップを追加 - Push後、DB操作前にCloud Runサービスを停止
2. TerminateConnectionsをWaitForProxyに名称変更・簡略化 - pg_terminate_backendのロジックを削除
3. SqlProxyの開始をStopCloudRun完了後に変更
4. 依存関係を適切に更新

背景

誰かがステージング環境にアクセスしているとDB削除ができずにデプロイが失敗する問題が長年ありました。Cloud SQLでは管理者権限を持つユーザーになることができないため、pg_terminate_backendで接続を切断することができませんでした。

解決策

Cloud Runサービス自体を停止することで、アプリケーションからのDB接続を確実に切断します。

Test plan

• ステージング環境へのデプロイが成功することを確認
• 誰かがアクセス中でもデプロイが成功することを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• Chores
  • データベース接続管理とサービス停止のシーケンスを最適化しました。
  • クラウドインフラストラクチャのデプロイメントワークフローを改善し、信頼性と接続処理の堅牢性を向上させました。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

ウォークスルー

Cloud Build ステージング パイプラインの接続管理フローを再構成します。Cloud Run サービスを停止するステップを追加し、直接的なデータベース接続の終了ロジックをアクティブな SQL Proxy 準備確認に置き換え、パイプライン ステップの依存関係を再編成します。

変更内容

コホート / ファイル	変更の要約
Cloud Build ステージング パイプラインの再構成 ​.cloudbuild/cloudbuild-staging.yaml	StopCloudRun ステップを新規追加してサービスのデータベース接続を切断。TerminateConnections ステップを削除し、Rails ベースの接続終了ロジックを廃止。WaitForProxy ステップを追加して Cloud SQL Proxy の準備状況をポーリング。SqlProxy が StopCloudRun の完了を待つよう設定。DeleteDB を TerminateConnections から WaitForProxy への依存に変更。

推定コードレビュー工数

🎯 3 (標準的) | ⏱️ 約20分

• Cloud Run サービス停止ロジックと Proxy 準備確認フローの動作確認
• パイプライン ステップ間の依存関係と実行順序の妥当性検証
• タイムアウト値（30秒）とポーリング間隔の適切性確認

関連する可能性のあるプルリクエスト

• ステージング環境のデプロイ時にデータベース削除エラーを修正 #9062 — 同じパイプラインの接続終了シーケンスとDeleteDB の依存関係を変更。TerminateConnections ステップの追加と異なり、本PR では新しい StopCloudRun/WaitForProxy フローに置き換え。
• Cloud Build: エラー出力を表示するように修正 #9351 — TerminateConnections ステップの直接的な変更対象。本PR でステップが削除・置き換えされる。
• Cloud Build: DBMigrateにAPP_HOST_NAMEを追加 #9353 — 同じパイプラインの複数ステップを変更。本PR の構造的変更とステップ順序の変更と重複・競合の可能性あり。

推奨ラベル

雑事

推奨レビュアー

• okuramasafumi

詩

🐰 クラウドの流れを整えて
ステップの順序 優雅に
サーバー停止して繋がり切れ
Proxy 待ちて安らかに
データベース削除 静かに

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/stop-cloudrun-before-db-delete

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7cd98dd and 918f900.

📒 Files selected for processing (1)

• .cloudbuild/cloudbuild-staging.yaml (3 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}