Skip to content

docs: record Rust build-speed experiments and correct ring advice - #1869

Merged
JacobStephens2 merged 3 commits into
mainfrom
issue-1868
Nov 9, 2026
Merged

JacobStephens2 merged 3 commits into
mainfrom
issue-1868

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

Summary

 docs/research/rust-build-profile.md
- Suggest ring alone; list linker and Cranelift as untried
+ Explain axoupdater's unified TLS features; cite #203, #1592 and Cargo.toml
+ Record the eight build experiments, host-specific conclusions and limits

Measurements come from #1868; they were not rerun for this PR. The original Method and timing table are preserved. Only the research note changes.

Closes #1868

Evidence

Execution evidence

  • rg -n '^## Experiments$' docs/research/rust-build-profile.md
    • Before: no matches; exit 1.
    • After: 73:## Experiments; exit 0.
  • /usr/bin/time -p scripts/affected-tests main: first run failed after 731.85 s including compilation. Nextest ran 195/3,927 tests: 194 passed, one failed, 91 skipped; fail-fast left 3,732 unrun. The failure was ci::actions_jobs_survive_a_check_runs_403_and_red_ci_gets_a_repair, where starting the fake agent failed with Unit thirdshift-session-2547107-0-harness.scope was already loaded or has a fragment file.
  • cargo test --test ci actions_jobs_survive_a_check_runs_403_and_red_ci_gets_a_repair -- --exact --nocapture: passed, 1 test, 5.62 s (plus 1.82 s build).
  • Final validation: CI's Rust job passed on the final head 1ae80a03: 3,927 tests passed, 91 skipped, 1,214.512 s test execution plus 70 s test compilation. Formatting, clippy and release-generator checks passed; site and Console CI passed too.
  • A second local /usr/bin/time -p scripts/affected-tests main ran with the build cached and was stopped by session ID after CI passed on that exact head, reusing successful CI as docs/testing.md directs. No test failures were reported before the intentional interruption (exit 130).
  • git diff --check main...HEAD: passed (exit 0).

Review coverage

Independent thirdshift-code-review reviewers compared against main and checked the final wording correction. Standards: 1 glossary finding, fixed and verified. Spec: 0 findings. Both reviewers read the only changed file, docs/research/rust-build-profile.md. Changed files left unread: none. Generated-output exclusions: none.

Unaddressed findings

Standards

None. The glossary finding was addressed in 1ae80a03.

Spec

None.

Security

None. The change is the research note only. It does not change the updater, TLS features, toolchain pin, or Cargo config.

Security review outcome

No unaddressed introduced findings.

Merge Danger

Door: two-way

The documentation edit can be reverted.

Blast Radius: documentation

Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit 9283d0b into main Nov 9, 2026
4 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1868 branch November 9, 2026 20:40
@JacobStephens2 JacobStephens2 mentioned this pull request Nov 10, 2026
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.

Record the Rust build-speed experiments and correct the ring suggestion in the build profile note

1 participant