Skip to content

Use debugserver for code intel worker health - #1421

Merged
eseliger merged 2 commits into
mainfrom
prepare-remove-pciw-health-port
Sep 6, 2026
Merged

Use debugserver for code intel worker health#1421
eseliger merged 2 commits into
mainfrom
prepare-remove-pciw-health-port

Conversation

@eseliger

@eseliger eseliger commented Sep 5, 2026

Copy link
Copy Markdown
Member

The precise and syntactic code intel workers already report readiness from their debugservers on port 6060. Point both Docker healthchecks there and remove the obsolete syntactic worker address configuration, aligning Docker Compose with the Kubernetes deployments.

The precise code intel worker already exposes readiness on its debugserver. Point the Compose healthcheck there so the redundant application health listener can be removed.

Amp-Thread-ID: https://ampcode.com/threads/T-01a0711c-d0c8-73ba-a343-5d5f8b76841b
Co-authored-by: Amp <amp@ampcode.com>
@eseliger eseliger changed the title chore/deploy: use debugserver for precise worker health Use debugserver for precise worker health Sep 5, 2026
@eseliger
eseliger marked this pull request as ready for review September 5, 2026 11:04
@eseliger
eseliger requested a review from a team September 5, 2026 11:04
@eseliger
eseliger enabled auto-merge (squash) September 5, 2026 11:04
The syntactic code intel worker already exposes readiness on its debugserver. Point the Compose healthcheck there so the redundant application health listener can be removed.

Amp-Thread-ID: https://ampcode.com/threads/T-01a0711c-d0c8-73ba-a343-5d5f8b76841b
Co-authored-by: Amp <amp@ampcode.com>
@eseliger eseliger changed the title Use debugserver for precise worker health Use debugserver for code intel worker health Sep 5, 2026
@eseliger
eseliger merged commit e01627a into main Sep 6, 2026
2 checks passed
@eseliger
eseliger deleted the prepare-remove-pciw-health-port branch September 6, 2026 05:30
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