Skip to content

ci: use free runner for aarch64-gnu-llvm-19-1 PR job #141767

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 2, 2025

Conversation

marcoieni
Copy link
Member

@marcoieni marcoieni commented May 30, 2025

By using a free runner, this PR job now takes 70 minutes instead of 60. I think it's fine because soon #136942 should reduce how long it takes to run this job anyway.

@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
@marcoieni
Copy link
Member Author

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 7ad2c36 with merge 83c1482

rust-bors bot added a commit that referenced this pull request May 30, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job

<!-- homu-ignore:start -->
<!--
If this PR is related to an unstable feature or an otherwise tracked effort,
please link to the relevant tracking issue here. If you don't know of a related
tracking issue or there are none, feel free to ignore this.

This PR will get automatically assigned to a reviewer. In case you would like
a specific user to review your work, you can assign it to them by using

    r? <reviewer name>
-->
r? `@ghost`
<!-- homu-ignore:end -->
@marcoieni
Copy link
Member Author

I made a mistake in running bors try, since this is a pr job 😅

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@marcoieni
Copy link
Member Author

r? @Kobzol

@marcoieni marcoieni marked this pull request as ready for review June 2, 2025 07:52
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 2, 2025
@Kobzol
Copy link
Contributor

Kobzol commented Jun 2, 2025

We need to reduce the costs here, so even taking the 60 -> 70 minute hit isn't the end of the world, although it is slightly annoying.

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 2, 2025

📌 Commit 7ad2c36 has been approved by Kobzol

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 2, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 2, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 2, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job
bors added a commit that referenced this pull request Jun 2, 2025
Rollup of 5 pull requests

Successful merges:

 - #141767 (ci: use free runner for aarch64-gnu-llvm-19-1 PR job)
 - #141858 (Fix typo in `StructuralPartialEq` docs)
 - #141865 (Optionally don't steal the THIR)
 - #141874 (add f16_epsilon and f128_epsilon diagnostic items)
 - #141904 (test-float-parse: apply `cfg(not(bootstrap))`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 2cb5b0d into rust-lang:master Jun 2, 2025
9 checks passed
@rustbot rustbot added this to the 1.89.0 milestone Jun 2, 2025
rust-timer added a commit that referenced this pull request Jun 2, 2025
Rollup merge of #141767 - marcoieni:free-pr-job, r=Kobzol

ci: use free runner for aarch64-gnu-llvm-19-1 PR job
jieyouxu pushed a commit to jieyouxu/rustc-dev-guide that referenced this pull request Jun 3, 2025
Rollup of 5 pull requests

Successful merges:

 - rust-lang/rust#141767 (ci: use free runner for aarch64-gnu-llvm-19-1 PR job)
 - rust-lang/rust#141858 (Fix typo in `StructuralPartialEq` docs)
 - rust-lang/rust#141865 (Optionally don't steal the THIR)
 - rust-lang/rust#141874 (add f16_epsilon and f128_epsilon diagnostic items)
 - rust-lang/rust#141904 (test-float-parse: apply `cfg(not(bootstrap))`)

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.

4 participants