Skip to content

use vpc egress to comm with cloud sql#58

Merged
zedimytch merged 4 commits into
masterfrom
fix/vpc-egress
Mar 17, 2026
Merged

use vpc egress to comm with cloud sql#58
zedimytch merged 4 commits into
masterfrom
fix/vpc-egress

Conversation

@zedimytch
Copy link
Copy Markdown
Collaborator

@zedimytch zedimytch 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.

@zedimytch zedimytch requested a review from Step7750 March 16, 2026 22:56
Comment thread deploy/reverse-watch-service.yaml Outdated
@zedimytch zedimytch self-assigned this Mar 16, 2026
@zedimytch zedimytch 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.

Comment thread deploy/reverse-watch-service.yaml
@zedimytch zedimytch merged commit 9b5cca9 into master Mar 17, 2026
3 checks passed
@zedimytch zedimytch 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