Skip to content

fix: start probe server before dependency init to prevent startup probe failures#296

Merged
luis5tb merged 1 commit into
RHEcosystemAppEng:mainfrom
luis5tb:worktree-agent-health
Jun 23, 2026
Merged

fix: start probe server before dependency init to prevent startup probe failures#296
luis5tb merged 1 commit into
RHEcosystemAppEng:mainfrom
luis5tb:worktree-agent-health

Conversation

@luis5tb

@luis5tb luis5tb commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

The probe server on port 8002 was starting after Redis, database, and service control initialization, so the Cloud Run startup probe had no endpoint to reach during that window. With only 35s allowed (3 failures × 10s + 5s delay), slow dependency connections caused the probe to fail and the deployment to abort.

Move probe server startup to the top of the lifespan so /health responds immediately, and increase failureThreshold from 3 to 18 (~185s window).

@luis5tb luis5tb force-pushed the worktree-agent-health branch from ea9af1b to f141e34 Compare June 15, 2026 07:46
@luis5tb luis5tb force-pushed the worktree-agent-health branch from f141e34 to eaec211 Compare June 22, 2026 13:24
…be failures

The probe server on port 8002 was starting after Redis, database, and
service control initialization, so the Cloud Run startup probe had no
endpoint to reach during that window. With only 35s allowed (3 failures
× 10s + 5s delay), slow dependency connections caused the probe to fail
and the deployment to abort.

Move probe server startup to the top of the lifespan so /health responds
immediately, and increase failureThreshold from 3 to 18 (~185s window).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@luis5tb luis5tb force-pushed the worktree-agent-health branch from eaec211 to 8347889 Compare June 22, 2026 14:54

@IlonaShishov IlonaShishov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@luis5tb luis5tb merged commit 7fff8f3 into RHEcosystemAppEng:main Jun 23, 2026
14 checks passed
IlonaShishov added a commit to IlonaShishov/google-lightspeed-agent that referenced this pull request Jun 24, 2026
The probe server was moved to the top of lifespan in PR RHEcosystemAppEng#296.
The rebase left a stale copy at the old location — remove it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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