ci(e2e): warm chat and platform URLs before QA.4 suite#13
Closed
Shaivpidadi wants to merge 1 commit intodevfrom
Closed
ci(e2e): warm chat and platform URLs before QA.4 suite#13Shaivpidadi wants to merge 1 commit intodevfrom
Shaivpidadi wants to merge 1 commit intodevfrom
Conversation
The previous warmup only pinged precheck, leaving the Render-hosted chat app and platform dashboard cold. Every spec starts with a nav to the chat app, so cold starts caused the entire QA.4 suite — including the PII decision-log test — to time out on `/login`. Warm all three Render services (precheck, chat, platform) in sequence so `pnpm test:e2e` starts against a hot staging environment. Each service is given up to 10 retries with a 15s sleep, matching the previous precheck-only behavior. Unblocks QA.4 acceptance criterion "Passes against staging".
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
Superseded by #12 (GOV-592), which applies the same warmup fix and also bumps navigation/test timeouts to absorb residual cold-start latency. Deferring to that PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/e2e.ymlto also ping the chat app and platform dashboard (currently only precheck is warmed).pii-decision-log.spec.ts— which has been failing the "passes against staging" acceptance criterion because the chat URL was cold-starting whenpage.goto('/')ran in the auth fixture.Context
Recent E2E runs on
dev(24855162725, 24843846754, 24843302754) all failed at the first step of every spec:The existing warmup pings only
$STAGING_PRECHECK_URL/api/v1/health, so the Render-hosted chat Next.js server was still cold when Playwright started. Warming it pre-test removes that class of flake and lets the real assertions (redact badge, decision log row) run.Test plan