Skip to content

use vpc egress to comm with cloud sql#58

Merged
hayesZach merged 4 commits intomasterfrom
fix/vpc-egress
Mar 17, 2026
Merged

use vpc egress to comm with cloud sql#58
hayesZach merged 4 commits intomasterfrom
fix/vpc-egress

Conversation

@hayesZach
Copy link
Copy Markdown
Collaborator

@hayesZach hayesZach commented Mar 16, 2026

Note

Medium Risk
Changes Cloud Run networking/egress and secret selection for DATABASE_HOST, which can break database connectivity if VPC or secret versions are misconfigured. Probe timing changes may also affect rollout behavior and readiness during cold starts.

Overview
Configures the reverse-watch Cloud Run service to use explicit VPC network interfaces with private-ranges-only egress to support private connectivity (e.g., to Cloud SQL).

Updates deployment wiring by switching DATABASE_HOST to a different secret key version and retuning the startupProbe to check more frequently with a higher failure threshold (same overall grace window).

Written by Cursor Bugbot for commit 72e5476. This will update automatically on new commits. Configure here.

@hayesZach hayesZach requested a review from Step7750 March 16, 2026 22:56
@hayesZach hayesZach self-assigned this Mar 16, 2026
@hayesZach hayesZach added the bug Something isn't working label Mar 16, 2026
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@hayesZach hayesZach merged commit 9b5cca9 into master Mar 17, 2026
3 checks passed
@hayesZach hayesZach deleted the fix/vpc-egress branch March 17, 2026 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants