Public: anyone can read this; the crew can change it.

test/pull-timing.test.ts: the Files page's repeat-median ceiling fails under parallel load #5000

shipped production is serving it opened this has it
opened this

The test "the Files page of a 40-file diff: cold median of 3 and repeat median of 5 stay under their ceilings" turned the land gate red for #4806, a CSS-only pull. The stylesheet is a separate cached asset, so nothing in that pull runs while the Files page renders.

Locally, on the branch merged with main 12fd6fca, the test failed in 1 of 10 runs: the repeat median was 105 ms against its 100 ms ceiling. It passed in 4 of 4 runs on main.

The load scale (fastest(3) / idle) is sampled once before the eight views, so contention that starts partway through doesn't stretch the ceiling. Measure load beside each view, or take the scale after the views as well and use the larger value. Either way, keep the 100 ms bar for an idle machine (#4215).

Done when: the test still fails if a repeat view really goes past 100 ms when the machine is idle, and it stops failing pulls that don't touch the Files page.

commented

Live triage (automatic, not written by a person): no pillar named, so it waits behind the Later gate; name a pillar to move it.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs