Skip to content

perf(ci): DEBUG=0 for the shard build (P1a, Refs #5486) - #5745

Merged
noahgift merged 1 commit into
mainfrom
perf/ci-debug0-p1a
Oct 19, 2026
Merged

noahgift merged 1 commit into
mainfrom
perf/ci-debug0-p1a

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

Operator 12:35 Madrid P1a (build once, link less): CARGO_PROFILE_DEV_DEBUG=0 and CARGO_PROFILE_TEST_DEBUG=0 in the seven env blocks of workspace-test-shard in ci/sections.yml. Nightly workflows keep line tables (untouched).

Measure (to post on merge): shard build 'Finished' time and target dir size per run vs MG 39227292002 (shard builds 6m16s/7m38s/7m31s; shard 1 7m33s). First run is cold for sccache (key change), so judge the second MG.
Gate: target dir <=25 GB/run; MG p90 <=35 min by 10-21.

Revert rule: MG p90 not lower after 2 MGs, revert.

🤖 Generated with Claude Code

…1a, Refs #5486)

Operator 12:35 Madrid P1a: build once, link less. Debug info dominates link time and
target-dir size; nightly keeps line tables. Seven env pairs in workspace-test-shard.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seat: aprender-w5382 (claude-sonnet-5-5, non-author) at fec85e1. Diff read in full: ci/sections.yml only, 7 hunks (14 lines) changing CARGO_PROFILE_TEST_DEBUG and CARGO_PROFILE_DEV_DEBUG from line-tables-only to 0, in the build/test/check steps. Nothing else changed. Grepped scripts, ci, crates, .github and Makefile: no guard or test pins line-tables-only; the one remaining mention in sections.yml is a comment. Panic locations do not depend on debuginfo; only symbolized backtrace line numbers are lost. APPROVE the diff. CI not complete when read (5 pending, 1 success), so diff-only.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seat 1, aprender-pace @fec85e142bc7e9c1d1f63ae5ee24b8c1dc8e182b

Non-author (0 aprender-pace trailers). Read the full delta: ci/sections.yml only, 7 docker run sites, CARGO_PROFILE_TEST_DEBUG and CARGO_PROFILE_DEV_DEBUG line-tables-only -> 0, symmetric, no site missed or half-changed. Compile-time lever only; no gate or test is added or removed.
Trade-off to name: debug=0 drops line tables, so a CI panic backtrace loses file:line frames (panic Location is unaffected). Fine if nobody reads CI backtraces.
NOT verified: CI (5 still in progress), the claimed build-time saving, that no test asserts on backtrace text.

@github-actions

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=5745 head=fec85e142bc7e9c1d1f63ae5ee24b8c1dc8e182b verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

@noahgift
noahgift enabled auto-merge October 19, 2026 11:57
@noahgift
noahgift added this pull request to the merge queue Oct 19, 2026
Merged via the queue into main with commit 81e7bc0 Oct 19, 2026
12 checks passed
@noahgift
noahgift deleted the perf/ci-debug0-p1a branch October 19, 2026 17:16
noahgift added a commit that referenced this pull request Oct 19, 2026
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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