Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CI: remove master job #124332

Merged
merged 6 commits into from Apr 29, 2024
Merged

CI: remove master job #124332

merged 6 commits into from Apr 29, 2024

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Apr 24, 2024 •
edited

It only had one job (pun intended), to publish the toolstate. We could probably do that at the end of auto builds instead, which is what is done in this PR.

r? @pietroalbini

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Apr 24, 2024
@Kobzol
Copy link
Contributor Author

Kobzol commented Apr 24, 2024

Ok, second attempt. I have moved toolstate publishing to the auto-success job, since it is already configured to run when the auto job succeeds. This conflicts with the change to unify the success/failure jobs though.

@pietroalbini
Copy link
Member

Not sure exactly why git decided to render the diff that way in the second commit 😅

r=me once the other PR is merged

@Kobzol
Copy link
Contributor Author

Kobzol commented Apr 24, 2024

Yeah that was super weird :D I moved the steps down from the anchor, but GH (also local git diff) decided to instead move the jobs up.

@Kobzol
Copy link
Contributor Author

Kobzol commented Apr 25, 2024 •
edited

Rebased since #124327 has been merged. I also added two new commits, so not approving immediately.

@Kobzol Kobzol mentioned this pull request Apr 25, 2024
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 25, 2024
@bors
Auto merge of rust-lang#124366 - Kobzol:remove-yaml-expansion, r=<try>
6629a69 
CI: remove `expand-yaml-anchors`

Opening for experiments.

Blocked on rust-lang#124332 (first 4 commits are from that PR).

r? `@ghost`
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 25, 2024
@bors
Auto merge of rust-lang#124366 - Kobzol:remove-yaml-expansion, r=<try>
9026eae 
CI: remove `expand-yaml-anchors`

Opening for experiments.

Blocked on rust-lang#124332 (first 4 commits are from that PR).

r? `@ghost`
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 25, 2024
@bors
Auto merge of rust-lang#124366 - Kobzol:remove-yaml-expansion, r=<try>
f42f9b6 
CI: remove `expand-yaml-anchors`

This PR unifies all CI outcome jobs in a single job, and then removes the `expand-yaml-anchors` tool, since it is no longer needed after this change.

Opening for experiments.

Blocked on rust-lang#124332 (first 4 commits are from that PR).

r? `@ghost`
@Kobzol
Copy link
Contributor Author

Kobzol commented Apr 25, 2024

@rustbot ready

@pietroalbini
Copy link
Member

@bors r+ rollup=iffy

@bors
Copy link
Contributor

bors commented Apr 27, 2024

📌 Commit 163ea33 has been approved by pietroalbini

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 Apr 27, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 27, 2024
@bors
Auto merge of rust-lang#124332 - Kobzol:toolstate-auto-build, r=pietr…
2c94366 
…oalbini

CI: remove `master` job

It only had one job (pun intended), to publish the toolstate. We could probably do that at the end of `auto` builds instead, which is what is done in this PR.

r? `@pietroalbini`
@bors
Copy link
Contributor

bors commented Apr 27, 2024

⌛ Testing commit 163ea33 with merge 2c94366...

@bors
Copy link
Contributor

bors commented Apr 27, 2024

💔 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 Apr 27, 2024
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Kobzol
Copy link
Contributor Author

Kobzol commented Apr 27, 2024

Added missing checkout step.

@pietroalbini
Copy link
Member

@bors r+ rollup=iffy

@bors
Copy link
Contributor

bors commented Apr 29, 2024

📌 Commit a700897 has been approved by pietroalbini

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 Apr 29, 2024
@bors
Copy link
Contributor

bors commented Apr 29, 2024

⌛ Testing commit a700897 with merge e27af29...

@bors
Copy link
Contributor

bors commented Apr 29, 2024

☀️ Test successful - checks-actions
Approved by: pietroalbini
Pushing e27af29 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 29, 2024
@bors bors merged commit e27af29 into rust-lang:master Apr 29, 2024
10 checks passed
@rustbot rustbot added this to the 1.80.0 milestone Apr 29, 2024
@Kobzol Kobzol deleted the toolstate-auto-build branch April 29, 2024 19:24
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (e27af29): 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)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)		 - - 0
Regressions ❌
(secondary)		 - - 0
Improvements ✅
(primary)		 -2.1% [-2.3%, -2.0%] 2
Improvements ✅
(secondary)		 - - 0
All ❌✅ (primary) -2.1% [-2.3%, -2.0%] 2

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)		 2.1% [2.1%, 2.1%] 1
Regressions ❌
(secondary)		 - - 0
Improvements ✅
(primary)		 - - 0
Improvements ✅
(secondary)		 - - 0
All ❌✅ (primary) 2.1% [2.1%, 2.1%] 1

Binary size

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

Bootstrap: 671.855s -> 673.188s (0.20%)
Artifact size: 315.98 MiB -> 315.98 MiB (0.00%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RalfJung RalfJung RalfJung left review comments

@pietroalbini pietroalbini Awaiting requested review from pietroalbini

Assignees

@pietroalbini pietroalbini

Labels
A-testsuite Area: The testsuite used to check the correctness of rustc 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-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.80.0
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@Kobzol @pietroalbini @bors @rust-log-analyzer @rust-timer @RalfJung @rustbot