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.

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

ci: use 20.04 on x86_64-gnu-nopt builder #79536

Merged
merged 1 commit into from
Dec 10, 2020

Conversation

davidtwco
Copy link
Member

@davidtwco davidtwco commented Nov 29, 2020

Switch the x86_64-gnu-nopt builder to use Ubuntu 20.04.

Ubuntu 20.04 has a more recent gdb version than Ubuntu 16.04 (9.1 vs 7.11.1), which is required for #77117, as 16.04's gdb 7.11.1 crashes in some cases with Split DWARF. x86_64-gnu-nopt is chosen because it runs compare modes, which is how Split DWARF testing is implemented in #77117.

I've not confirmed that the issue is resolved with gdb 9.1 (Feb 2020), but system was using gdb 9.2 (May 2020) and that was fine and it seems more likely to me that the bug was resolved between gdb 7.11.1 (May 2016) and gdb 9.1.

Updating a builder to use 20.04 was suggested by @Mark-Simulacrum in #77117 (comment). I'm not sure if this is the only change that is required - if more are necessary then I'm happy to do that.

r? @pietroalbini
cc @Mark-Simulacrum

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 29, 2020
@jyn514 jyn514 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 Nov 29, 2020
@pietroalbini
Copy link
Member

This should be enough!

@bors r+ rollup=iffy

@bors
Copy link
Contributor

bors commented Nov 30, 2020

📌 Commit 68e15c86aec3fd45f843c785f3669a02f85c7816 has been approved by pietroalbini

@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 Nov 30, 2020
@bors
Copy link
Contributor

bors commented Nov 30, 2020

⌛ Testing commit 68e15c86aec3fd45f843c785f3669a02f85c7816 with merge 33f0985b8eeeb8abcaacc6ce9d4120d222a583f8...

@bors
Copy link
Contributor

bors commented Nov 30, 2020

💔 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 Nov 30, 2020
@pietroalbini
Copy link
Member

Spurious network failure.

@bors retry

@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 Nov 30, 2020
@bors
Copy link
Contributor

bors commented Nov 30, 2020

⌛ Testing commit 68e15c86aec3fd45f843c785f3669a02f85c7816 with merge 970455fc2d1d28c7dddc78aa43dedc63939863b3...

@bors
Copy link
Contributor

bors commented Nov 30, 2020

💥 Test timed out

@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 Nov 30, 2020
@lqd
Copy link
Member

lqd commented Dec 1, 2020

maybe we can @bors retry this once more

@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 Dec 1, 2020
@pietroalbini
Copy link
Member

@bors

@pietroalbini
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Dec 1, 2020

⌛ Testing commit 68e15c86aec3fd45f843c785f3669a02f85c7816 with merge 85a07d14facf9aada1ef48cdbefd85daeafd7e48...

@bors
Copy link
Contributor

bors commented Dec 1, 2020

💔 Test failed - checks-actions

@bors bors removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 1, 2020
@bors bors added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 1, 2020
@pietroalbini
Copy link
Member

@bors retry gha outage

@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 Dec 2, 2020
@bors
Copy link
Contributor

bors commented Dec 2, 2020

⌛ Testing commit 68e15c86aec3fd45f843c785f3669a02f85c7816 with merge 37005e4d44e6daf6fcf60c874700b610ca647f60...

@bors
Copy link
Contributor

bors commented Dec 2, 2020

💥 Test timed out

@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 Dec 2, 2020
@davidtwco
Copy link
Member Author

Looks like that last failure wasn't spurious, attempted to address it in most recent push.

@davidtwco
Copy link
Member Author

@pietroalbini can you re-review?

This commit switches the x86_64-gnu-nopt builder to use Ubuntu 20.04,
which contains a more recent gdb version than Ubuntu 16.04 (newer gdb
versions fix a bug that Split DWARF can trigger, see
rust-lang#77177 for motivation). x86_64-gnu-nopt is chosen because
it runs compare modes, which is how Split DWARF testing is implemented
in rust-lang#77177.

Signed-off-by: David Wood <david@davidtw.co>
@pietroalbini
Copy link
Member

@bors r+ rollup=never

@bors
Copy link
Contributor

bors commented Dec 10, 2020

📌 Commit fb75c32 has been approved by pietroalbini

@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 Dec 10, 2020
@bors
Copy link
Contributor

bors commented Dec 10, 2020

⌛ Testing commit fb75c32 with merge 80cc2ec...

@bors
Copy link
Contributor

bors commented Dec 10, 2020

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

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 10, 2020
@bors bors merged commit 80cc2ec into rust-lang:master Dec 10, 2020
@rustbot rustbot added this to the 1.50.0 milestone Dec 10, 2020
@davidtwco davidtwco deleted the focal-fossa-ci branch December 10, 2020 17:50
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 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
Development

Successfully merging this pull request may close these issues.

7 participants