Skip to content

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented May 30, 2025

Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed here.

r? @marcoieni

try-job: dist-x86_64-linux*

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels May 30, 2025
@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit e28d0c7 with merge 31b9ec6

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from e28d0c7 to 318d1e9 Compare May 30, 2025 16:34
@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 318d1e9 with merge 6df9b4f

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from 318d1e9 to 65bb0d4 Compare May 30, 2025 17:38
@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 65bb0d4 with merge 841e227

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from 65bb0d4 to 033ea63 Compare May 30, 2025 18:36
@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

@bors2 try

Fucking submodules.

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 033ea63 with merge dce110a

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

@bors2 try cancel

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

Try build cancelled.
Cancelled workflows:

@Kobzol
Copy link
Member Author

Kobzol commented May 30, 2025

Nice, the alt job took only ~1.5h 🎉 Now let's see how long it takes on a free runner.

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 6e64f8f with merge 713ede4

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@mati865
Copy link
Member

mati865 commented May 30, 2025

IIRC results from this build are used for test jobs. So, they will likely slow down a bit but probably not by much.

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol
Copy link
Member Author

Kobzol commented May 31, 2025

Ok, looks like free runner is a no-go because of disk space, I guess that makes sense.

IIRC results from this build are used for test jobs. So, they will likely slow down a bit but probably not by much.

Thanks, I'll have to check that out.

@rust-bors
Copy link

rust-bors bot commented Jun 2, 2025

⌛ Trying commit eaaa846 with merge 336a518

@rust-bors
Copy link

rust-bors bot commented Jun 2, 2025

☀️ Try build successful (CI)
Build commit: 336a518 (336a51809f5af5b476831469f189e6457aa71ef3)

@Kobzol
Copy link
Member Author

Kobzol commented Jun 2, 2025

~1h 45m on Codebuild, ideal.

@Kobzol Kobzol marked this pull request as ready for review June 2, 2025 13:14
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 2, 2025
@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from eaaa846 to 8f22a50 Compare June 3, 2025 08:23
@Kobzol
Copy link
Member Author

Kobzol commented Jun 3, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented Jun 3, 2025

⌛ Trying commit 8f22a50 with merge 6a184ec

rust-bors bot added a commit that referenced this pull request Jun 3, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@marcoieni
Copy link
Member

this is great, thanks ❤️
Feel free to r=me if the timing is good

@rust-bors
Copy link

rust-bors bot commented Jun 3, 2025

☀️ Try build successful (CI)
Build commit: 6a184ec (6a184ec3874dd1909ca928da012736b935aedb3e)

@Kobzol
Copy link
Member Author

Kobzol commented Jun 3, 2025

2h 45m. A bit borderline, but we can always switch to the 8-core runner if it becomes too much. And at least it shouldn't be the bottleneck anymore.

@bors r=marcoieni

@bors
Copy link
Collaborator

bors commented Jun 3, 2025

📌 Commit 8f22a50 has been approved by marcoieni

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 Jun 3, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 5, 2025
… r=marcoieni

Do not run PGO/BOLT in x64 Linux alt builds

Should unblock rust-lang#131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
bors added a commit that referenced this pull request Jun 5, 2025
Rollup of 8 pull requests

Successful merges:

 - #140638 (UnsafePinned: also include the effects of UnsafeCell)
 - #141272 (modularize the config module bootstrap)
 - #141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - #141870 (Fix broken link to rustc_type_ir module in serialization docs)
 - #141938 (update rust offload bootstrap)
 - #141962 (rustc-dev-guide subtree update)
 - #141965 (`tests/ui`: A New Order [3/N])
 - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Jun 5, 2025
Rollup of 6 pull requests

Successful merges:

 - #140638 (UnsafePinned: also include the effects of UnsafeCell)
 - #141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - #141938 (update rust offload bootstrap)
 - #141962 (rustc-dev-guide subtree update)
 - #141965 (`tests/ui`: A New Order [3/N])
 - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 2c5a5fe into rust-lang:master Jun 5, 2025
10 checks passed
@rustbot rustbot added this to the 1.89.0 milestone Jun 5, 2025
rust-timer added a commit that referenced this pull request Jun 5, 2025
Rollup merge of #141777 - Kobzol:dist-linux-alt-no-pgo-bolt, r=marcoieni

Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? ``@marcoieni``

try-job: `dist-x86_64-linux*`
@Kobzol Kobzol deleted the dist-linux-alt-no-pgo-bolt branch June 5, 2025 17:15
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Jun 6, 2025
Rollup of 6 pull requests

Successful merges:

 - rust-lang/rust#140638 (UnsafePinned: also include the effects of UnsafeCell)
 - rust-lang/rust#141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - rust-lang/rust#141938 (update rust offload bootstrap)
 - rust-lang/rust#141962 (rustc-dev-guide subtree update)
 - rust-lang/rust#141965 (`tests/ui`: A New Order [3/N])
 - rust-lang/rust#141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
github-actions bot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Jun 9, 2025
Rollup of 6 pull requests

Successful merges:

 - rust-lang/rust#140638 (UnsafePinned: also include the effects of UnsafeCell)
 - rust-lang/rust#141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - rust-lang/rust#141938 (update rust offload bootstrap)
 - rust-lang/rust#141962 (rustc-dev-guide subtree update)
 - rust-lang/rust#141965 (`tests/ui`: A New Order [3/N])
 - rust-lang/rust#141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
tautschnig pushed a commit to model-checking/verify-rust-std that referenced this pull request Jun 17, 2025
…iaskrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#140638 (UnsafePinned: also include the effects of UnsafeCell)
 - rust-lang#141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - rust-lang#141938 (update rust offload bootstrap)
 - rust-lang#141962 (rustc-dev-guide subtree update)
 - rust-lang#141965 (`tests/ui`: A New Order [3/N])
 - rust-lang#141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants