Skip to content

Add the x86_64-gnu-stable builder #86098

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 3 commits into from
Jun 10, 2021
Merged

Conversation

pietroalbini
Copy link
Member

@pietroalbini pietroalbini commented Jun 7, 2021

During the 1.52 release process we had to deal with some commits that passed the test suite on the nightly branch but failed on the beta or stable branch. In that case it was due to some UI tests including the channel name in the output, but other changes might also be dependent on the channel.

This commit adds a new CI job that runs the Linux x86_64 test suite with the stable branch, ensuring nightly changes also work as stable. To ensure the new job works the following other changes are present:

  • The ui-fulldeps/session-derive-errors.rs test has been disabled on beta and stable, which required adding support for // ignore-{channel} and // only-{channel}.
  • The rustdoc/intra-doc/field.rs has been fixed.

r? @Mark-Simulacrum
fixes rust-lang/release-team#11

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 7, 2021
@pietroalbini
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

⌛ Trying commit db3b8032b9b02bbbbaad73936fbb28d853d18ebc with merge e6e6ad566431f5235b25806487450ceea90ec514...

@pietroalbini
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

⌛ Trying commit 97438ae3d5a1087a4184529c5198adcc4ff59f2e with merge 0308500f867a0d7159179cd9f66f66e0f06b967a...

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

💔 Test failed - checks-actions

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 7, 2021
@pietroalbini
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

⌛ Trying commit f1982ce649e08b16a49b5e3bb4d769970b2b0d05 with merge 3dc547091ab35053375bf40de567df6db7819869...

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

💔 Test failed - checks-actions

The session-derive-errors test ensures the internal SessionDiagnostic
derive macro outputs the right error messages when misused.

The macro relies on the proc_macro2 crate though, which changes its span
behavior depending on whether the channel is nightly or not. This caused
test failures when bumping the channel from nightly to beta/stable.

Since SessionDiagnostic is internal-only we don't care about its
diagnostics quality outside of nightly, as the compiler itself is
developed on nightly. Thus the easiest solution is to ignore that test
on the beta and stable channels.

This also implements `// only-{channel}` and `// ignore-{channel}` in
compiletest to properly support the change.
@pietroalbini
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

⌛ Trying commit 78b70bc9da3c4735da7bbfa0a05ee6946a94a5a1 with merge 11a59a658ff1c970d5a63e559c67baa9c536e1dd...

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

💔 Test failed - checks-actions

During the 1.52 release process we had to deal with some commits that
passed the test suite on the nightly branch but failed on the beta or
stable branch. In that case it was due to some UI tests including the
channel name in the output, but other changes might also be dependent on
the channel.

This commit adds a new CI job that runs the Linux x86_64 test suite with
the stable branch, ensuring nightly changes also work as stable.
@pietroalbini
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

⌛ Trying commit c7981b390bad3c9961ca09d5aecd5dbf2e5c1977 with merge dce2a5127ffe1878ea2e20443b7ef625e23c35da...

@bors
Copy link
Collaborator

bors commented Jun 7, 2021

☀️ Try build successful - checks-actions
Build commit: dce2a5127ffe1878ea2e20443b7ef625e23c35da (dce2a5127ffe1878ea2e20443b7ef625e23c35da)

@pietroalbini
Copy link
Member Author

Ok this should be ready for review.

r? @Mark-Simulacrum

@pietroalbini pietroalbini added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jun 7, 2021
@Mark-Simulacrum
Copy link
Member

This looks OK to me. Hopefully we won't have cases of someone gating behavior on beta vs. stable...

@bors r+ rollup=iffy

@bors
Copy link
Collaborator

bors commented Jun 9, 2021

📌 Commit 12d37e6 has been approved by Mark-Simulacrum

@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 9, 2021
@bors
Copy link
Collaborator

bors commented Jun 10, 2021

⌛ Testing commit 12d37e6 with merge c622840...

@bors
Copy link
Collaborator

bors commented Jun 10, 2021

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing c622840 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jun 10, 2021
@bors bors merged commit c622840 into rust-lang:master Jun 10, 2021
@rustbot rustbot added this to the 1.54.0 milestone Jun 10, 2021
@pietroalbini pietroalbini deleted the test-stable branch June 10, 2021 20:36
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run test suite with --release-channel=stable
6 participants