Repository navigation
54-ci: CI loose ends (perf-budget parts, script-guard, net-handshake) - #484
Merged
Merged
Conversation
- one job took 53-73 min against its 75 min limit; release.yml's run for 1.45.0 timed out with three perf-budget runs started by one tag push - the push / PR / release.yml triggers now run the --part 1/3..3/3 matrix deps.yml already used (~20-25 min a part); deps.yml's per-part calls are unchanged; per-part limit 50 min - no required check names the job (only deps-gate / pc-battery on deps) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…80 s (54 X10)
Since the scripts moved into the sandbox guest, globalThis.__slowRuns was the
guest's global, so the page always read 1; and { timeout: 180000 } sat in the
arg slot of waitForFunction, so the wait ended at the 30 s default. On a
~4 fps runner the 30 slow runs plus the runaway node's per-frame loop limit
take longer than that: red on CI since 1.44.0. The counter now rides the
target box's position.z, the options are the third argument, the runaway
node is removed before the section, and the label carries runs + seconds.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… out of quarantine (54 X11) The third peer's "C received the scene" read C's count after a fixed wait and was red on CI and under load with C partway (1/3/4 of 5). It now waits for A to have received C's getobjects (premise), then polls up to 90 s comparing C with A's current count; a red carries the C/A trail over time. A's five boxes are polled instead of a 1 s wait, and "C holds open conns" is bounded at 60 s. On a GPU host the trail shows C at 0/5 and 4/5 for up to 12 s before 5/5 (3/3 green), the window a fixed read raced. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
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
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.
Roadmap 54 rows X10 and X11 (X9 is in the cloud repo).
X10 — perf-budget under the job limit.
perf-budget.ymlalways runs as a 3-part matrix (perf-budget 1/3..3/3, 50 min each), the splitdeps.ymlalready uses. Single-job runs took 53-73 min against the 75 min limit; one tag push in 1.45.0 timed out. No required check names this job.X10 — e2e-smoke
script-guard"a slow node is paused". Test-only fix. Since scripts run in the sandbox guest, the test'sglobalThis.__slowRunscounter was the guest's, so the page read 1.{ timeout: 180000 }was passed as waitForFunction's arg, so the real limit was 30 s. The counter now rides the target box'sposition.z, the options are the third argument, and the runaway node is removed before the section.X11 — net-handshake "C received the scene": the test waits for A to have received C's
getobjects, then polls up to 90 s against A's current count (a red prints the C/A trail). The quarantine entry is removed.Proof
🤖 Generated with Claude Code