Skip to content

fix/dev: Fix Orb setup to avoid false cold-start failure - #1877

Merged
marcleblanc2 merged 1 commit into
mainfrom
orb/drop-docs-health-check
Sep 11, 2026
Merged

fix/dev: Fix Orb setup to avoid false cold-start failure#1877
marcleblanc2 merged 1 commit into
mainfrom
orb/drop-docs-health-check

Conversation

@marcleblanc2

@marcleblanc2 marcleblanc2 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Why

amp orb services ensure printed docs: NOT RESPONDING (readiness check timed out after 10000ms) on every fresh orb, even though the site came up fine ~30s later. next dev opens its port immediately, but contentlayer generation (521 docs) plus the first-page compile exceed Amp's fixed 10s readiness window, and .amp/services.yaml has no configurable timeout.

Change

Remove health: / from .amp/services.yaml. Amp now treats the service as ready when the port accepts a connection, which next dev does within a couple of seconds. The browser waits for the first compile instead of ensure reporting a false failure.

Tradeoff: ensure now proves the server is listening, not that a page rendered. A build error shows up in the browser or amp orb service logs docs instead.

Verified

In an orb: ensure now reports docs: started, port 31414, listening in ~8s, and the portal URL returns 200 for / and /cody.

Amp threads

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sourcegraph-docs Ready Ready Preview Sep 11, 2026 12:53am UTC

Request Review

@marcleblanc2 marcleblanc2 changed the title orb: drop docs health check to avoid false cold-start failure orb: Drop docs health check to avoid false cold-start failure Sep 9, 2026
@marcleblanc2 marcleblanc2 changed the title orb: Drop docs health check to avoid false cold-start failure chore/dev: Drop docs health check from orb setup to avoid false cold-start failure Sep 9, 2026
@marcleblanc2 marcleblanc2 changed the title chore/dev: Drop docs health check from orb setup to avoid false cold-start failure fix/dev: Fix Orb setup to avoid false cold-start failure Sep 9, 2026
@marcleblanc2
marcleblanc2 enabled auto-merge (squash) September 10, 2026 00:33
@marcleblanc2
marcleblanc2 requested review from a team and removed request for a team September 10, 2026 07:49
…d-start failure

next dev opens its port immediately, but contentlayer generation plus the
first-page compile take ~30s, longer than Amp's fixed 10s readiness window.
With health: / every fresh orb printed NOT RESPONDING even though the site
came up fine. Port-open readiness reports success as soon as the dev server
listens; the browser waits for the first compile instead.

Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-01a07e6b-4d78-7569-bb05-289a9c52ec9f
@marcleblanc2
marcleblanc2 force-pushed the orb/drop-docs-health-check branch from da63fc8 to c063625 Compare September 11, 2026 00:51
@marcleblanc2
marcleblanc2 merged commit ce76558 into main Sep 11, 2026
2 checks passed
@marcleblanc2
marcleblanc2 deleted the orb/drop-docs-health-check branch September 11, 2026 00:53
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