Skip to content

ci(perf): loosen the HTTPS churn-ratio backstop for Intel runners - #316

Merged
gg582 merged 1 commit into
devfrom
ci/https-gates-intel
Oct 5, 2026
Merged

gg582 merged 1 commit into
devfrom
ci/https-gates-intel

Conversation

@gg582

@gg582 gg582 commented Oct 5, 2026

Copy link
Copy Markdown
Member

Follow-up to #314. The first dev gate run after the merge (37301931499) landed on a new Intel Xeon Platinum 8573C runner and failed on healthy code. Its churn ratio was 0.071 against the 0.08 backstop. RTT was 0.08 ms and every other gate passed.

The AMD runs measured 0.11–0.13, so I assumed the ratio was hardware-neutral. Across CPU vendors it isn't: a regressed AMD tree measures 0.052 and a healthy Intel one 0.071, so no absolute backstop can separate them.

All 8 recorded CI runs pass 10/10. The tree with the #307 fix disabled still fails (RTT, churn). scripts/ci unit tests pass.

The first gate run on dev after c7f71d3 landed on an Intel Xeon Platinum
8573C and failed on healthy code: churn ratio 0.071 against the 0.08
backstop (RTT 0.08 ms, every other gate fine).  Plaintext accepts are
relatively faster there, so the ratio is not as hardware-neutral across
CPU vendors as the AMD-only data suggested: a regressed AMD tree measures
0.052 and a healthy Intel one 0.071, so no absolute backstop separates
them.

Drop the churn-ratio backstop to 0.04 (large drops only) and leave the
rest to the same-CPU check.  #307-class regressions stay caught by the
RTT gate on any CPU (0.08 -> 43 ms).  Record the Intel run in the history;
tests cover it and the same-CPU churn-ratio path.
@gg582
gg582 merged commit 610fe55 into dev Oct 5, 2026
15 checks passed
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.

1 participant