⏱️ ci: Give the E2E Redis Transport Lane Its Own Timing Budget (#14907)

The redis transport lane inherits the mock config's 10s assertion budget and
2 CI retries, both of which were tuned against the in-memory stream store.
Every stream event here crosses a real Redis round-trip, and the pause/resume
and rehydrate scenarios replay an entire job, so the same waits run much
closer to their budget than the memory shards ever do.

The suite is serial (`workers: 1`), so this is per-operation latency rather
than worker contention — which is why this lane reports flaky runs the memory
shards do not.

Raise the default assertion budget to 20s and allow one more retry for this
config only. The memory shards keep their current settings.
This commit is contained in:
Danny Avila 2026-08-16 21:28:16 -04:00 committed by GitHub
parent 107050396e
commit e34d83bf7a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -4,6 +4,16 @@ import mockConfig from './playwright.config.mock';
/** Browser scenarios whose behavior crosses the generation stream-store boundary. */
export default defineConfig({
...mockConfig,
/**
* Every stream event crosses a real Redis round-trip on this lane, so waits that
* sit comfortably against the in-memory store run much closer to their budget
* pause/resume and rehydrate paths most of all, since they replay a whole job.
* The suite is serial (`workers: 1`), so this is per-operation latency rather
* than contention, and it is why this lane reports flaky runs the memory shards
* never produce. Give it a larger default assertion budget and one more retry.
*/
retries: process.env.CI ? 3 : 0,
expect: { ...mockConfig.expect, timeout: 20_000 },
testMatch: [
/completion\.spec\.ts/,
/deferred-tools-hitl\.spec\.ts/,