Skip to content

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented Sep 19, 2025

Just set it to true, rather than having different default values on CI and locally, and then only deny true on our own CI, not elsewhere.

Closes: #146768

r? @jieyouxu

@rustbot
Copy link
Collaborator

rustbot commented Sep 19, 2025

This PR modifies src/bootstrap/src/core/config.

If appropriate, please update CONFIG_CHANGE_HISTORY in src/bootstrap/src/utils/change_tracker.rs.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Sep 19, 2025
@RalfJung
Copy link
Member

Thanks a lot :)

@rust-log-analyzer

This comment has been minimized.

@Kobzol Kobzol force-pushed the download-ci-llvm-default-value branch from b6e9186 to e3d682a Compare September 19, 2025 14:27
@rust-log-analyzer

This comment has been minimized.

@Kobzol Kobzol force-pushed the download-ci-llvm-default-value branch from e3d682a to 4b75514 Compare September 19, 2025 18:12
Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

@jieyouxu
Copy link
Member

@bors r+ rollup=never

@bors
Copy link
Collaborator

bors commented Sep 20, 2025

📌 Commit 4b75514 has been approved by jieyouxu

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 20, 2025
bors added a commit that referenced this pull request Sep 20, 2025
…ouxu

Simplify default value of `download-ci-llvm`

Just set it to true, rather than having different default values on CI and locally, and then only deny `true` on our own CI, not elsewhere.

Closes: #146768

r? `@jieyouxu`
@bors
Copy link
Collaborator

bors commented Sep 20, 2025

⌛ Testing commit 4b75514 with merge bdcf701...

@rust-log-analyzer
Copy link
Collaborator

The job aarch64-gnu-llvm-20-1 failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors
Copy link
Collaborator

bors commented Sep 20, 2025

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Sep 20, 2025
@RalfJung
Copy link
Member

@bors retry curl: (28) Operation too slow. Less than 10 bytes/sec transferred the last 30 seconds

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 20, 2025
@bors
Copy link
Collaborator

bors commented Sep 20, 2025

⌛ Testing commit 4b75514 with merge b00998a...

@bors
Copy link
Collaborator

bors commented Sep 20, 2025

☀️ Test successful - checks-actions
Approved by: jieyouxu
Pushing b00998a to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Sep 20, 2025
@bors bors merged commit b00998a into rust-lang:master Sep 20, 2025
11 checks passed
@rustbot rustbot added this to the 1.92.0 milestone Sep 20, 2025
Copy link
Contributor

What is this? This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.

Comparing e4b5219 (parent) -> b00998a (this PR)

Test differences

Show 2 test diffs

2 doctest diffs were found. These are ignored, as they are noisy.

Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard b00998aaa50a78b1e45fb107a6c0cfd3f1dc44dd --output-dir test-dashboard

And then open test-dashboard/index.html in your browser to see an overview of all executed tests.

Job duration changes

  1. dist-aarch64-linux: 6187.1s -> 8911.0s (44.0%)
  2. aarch64-apple: 5602.8s -> 5037.6s (-10.1%)
  3. dist-aarch64-apple: 6750.9s -> 6079.3s (-9.9%)
  4. dist-various-2: 2195.9s -> 2407.0s (9.6%)
  5. x86_64-gnu-aux: 6800.4s -> 6307.2s (-7.3%)
  6. x86_64-msvc-ext3: 6699.7s -> 6242.0s (-6.8%)
  7. dist-apple-various: 3455.3s -> 3659.2s (5.9%)
  8. dist-x86_64-apple: 7129.6s -> 7443.7s (4.4%)
  9. test-various: 4413.8s -> 4607.2s (4.4%)
  10. dist-aarch64-msvc: 5937.0s -> 5683.8s (-4.3%)
How to interpret the job duration changes?

Job durations can vary a lot, based on the actual runner instance
that executed the job, system noise, invalidated caches, etc. The table above is provided
mostly for t-infra members, for simpler debugging of potential CI slow-downs.

@Kobzol Kobzol deleted the download-ci-llvm-default-value branch September 20, 2025 11:58
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (b00998a): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

This benchmark run did not return any relevant results for this metric.

Cycles

This benchmark run did not return any relevant results for this metric.

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 472.018s -> 471.747s (-0.06%)
Artifact size: 389.98 MiB -> 389.94 MiB (-0.01%)

Muscraft pushed a commit to Muscraft/rust that referenced this pull request Sep 24, 2025
…e, r=jieyouxu

Simplify default value of `download-ci-llvm`

Just set it to true, rather than having different default values on CI and locally, and then only deny `true` on our own CI, not elsewhere.

Closes: rust-lang#146768

r? `@jieyouxu`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bootstrap seems to default to downloading the LLVM submodule
7 participants