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

Update rustc-guide #63705

Merged
merged 1 commit into from
Aug 22, 2019
Merged

Update rustc-guide #63705

merged 1 commit into from
Aug 22, 2019

Conversation

mark-i-m
Copy link
Member

Should fix toolstate failure

r? @ehuss

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 19, 2019
@rust-highfive
Copy link
Collaborator

The job LinuxTools of your PR failed (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
2019-08-19T16:07:47.8054701Z ##[command]git remote add origin https://github.com/rust-lang/rust
2019-08-19T16:07:47.8243078Z ##[command]git config gc.auto 0
2019-08-19T16:07:47.8308303Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2019-08-19T16:07:47.8355913Z ##[command]git config --get-all http.proxy
2019-08-19T16:07:47.8471640Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/63705/merge:refs/remotes/pull/63705/merge
---
2019-08-19T16:08:22.5832585Z do so (now or later) by using -b with the checkout command again. Example:
2019-08-19T16:08:22.5832777Z 
2019-08-19T16:08:22.5833129Z   git checkout -b <new-branch-name>
2019-08-19T16:08:22.5833490Z 
2019-08-19T16:08:22.5838226Z HEAD is now at d093c8c79 Merge 0100f8b84c0bf7052759ccf370ed29359aa7bd64 into cdff9189556bb7de2b9a8a72344c9d8ec6099fcd
2019-08-19T16:08:22.5961418Z ##[section]Finishing: Checkout
2019-08-19T16:08:22.5967098Z ##[section]Starting: Decide whether to run this job
2019-08-19T16:08:22.5969996Z Task         : Bash
2019-08-19T16:08:22.5970042Z Description  : Run a Bash script on macOS, Linux, or Windows
2019-08-19T16:08:22.5970088Z Version      : 3.151.3
2019-08-19T16:08:22.5970152Z Author       : Microsoft Corporation
2019-08-19T16:08:22.5970152Z Author       : Microsoft Corporation
2019-08-19T16:08:22.5970215Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
2019-08-19T16:08:22.5970266Z ==============================================================================
2019-08-19T16:08:22.7137946Z Generating script.
2019-08-19T16:08:22.7165784Z ========================== Starting Command Output ===========================
2019-08-19T16:08:22.7182837Z [command]/bin/bash --noprofile --norc /home/vsts/work/_temp/9c152950-48ee-40ec-bb1a-b1c74ebeffbd.sh
2019-08-19T16:08:23.0065314Z Executing the job since submodules are updated
2019-08-19T16:08:23.0145603Z ##[section]Finishing: Decide whether to run this job
2019-08-19T16:08:23.0154732Z ==============================================================================
2019-08-19T16:08:23.0154816Z Task         : Bash
2019-08-19T16:08:23.0154851Z Description  : Run a Bash script on macOS, Linux, or Windows
2019-08-19T16:08:23.0155028Z Version      : 3.151.3
---
2019-08-19T18:25:44.6160709Z thread panicked while panicking. aborting.
2019-08-19T18:25:44.8629497Z [2019-08-19T18:25:44Z ERROR rls::server] Can't read message
2019-08-19T18:25:44.8630751Z thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 32, kind: BrokenPipe, message: "Broken pipe" }', src/libcore/result.rs:1084:5
2019-08-19T18:25:44.8637766Z note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
2019-08-19T18:25:44.8638593Z error: process didn't exit successfully: `/checkout/obj/build/x86_64-unknown-linux-gnu/stage2-tools/x86_64-unknown-linux-gnu/release/deps/client-4789b7d22e4fb0f9` (signal: 4, SIGILL: illegal instruction)
2019-08-19T18:25:44.8642753Z 
2019-08-19T18:25:44.8643533Z command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "test" "--target" "x86_64-unknown-linux-gnu" "-Zbinary-dep-depinfo" "-j" "2" "--release" "--locked" "--color" "always" "--manifest-path" "/checkout/src/tools/rls/Cargo.toml" "--features" "rustc-workspace-hack/all-static" "--"
2019-08-19T18:25:44.8643764Z expected success, got: exit code: 101
2019-08-19T18:25:44.8643854Z 
---
2019-08-19T18:28:47.5590371Z Verifying status of embedded-book...
2019-08-19T18:28:47.5602004Z Verifying status of rustc-guide...
2019-08-19T18:28:47.5613585Z This PR updated 'src/doc/rustc-guide', verifying if status is 'test-pass'...
2019-08-19T18:28:47.5622321Z 
2019-08-19T18:28:47.5630252Z ⚠️ We detected that this PR updated 'rustc-guide', but its tests failed.
2019-08-19T18:28:47.5630349Z 
2019-08-19T18:28:47.5630663Z If you do intend to update 'rustc-guide', please check the error messages above and
2019-08-19T18:28:47.5630739Z commit another update.
2019-08-19T18:28:47.5630767Z 
2019-08-19T18:28:47.5631024Z If you do NOT intend to update 'rustc-guide', please ensure you did not accidentally
2019-08-19T18:28:47.5631278Z change the submodule at 'src/doc/rustc-guide'. You may ask your reviewer for the
2019-08-19T18:28:47.5631346Z proper steps.
2019-08-19T18:28:47.5641835Z   local time: Mon Aug 19 18:28:47 UTC 2019
2019-08-19T18:28:47.7448393Z   network time: Mon, 19 Aug 2019 18:28:47 GMT
2019-08-19T18:28:47.7452906Z == end clock drift check ==
2019-08-19T18:28:47.7452906Z == end clock drift check ==
2019-08-19T18:28:48.7124105Z ##[error]Bash exited with code '3'.
2019-08-19T18:28:48.7171964Z ##[section]Starting: Checkout
2019-08-19T18:28:48.7173576Z ==============================================================================
2019-08-19T18:28:48.7173618Z Task         : Get sources
2019-08-19T18:28:48.7173658Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@ehuss
Copy link
Contributor

ehuss commented Aug 19, 2019

Error: there are broken links
Caused By: There was an error while fetching "http://octarineparrot.com/assets/mrfloya-thesis-ba.pdf", https://octarineparrot.com/assets/mrfloya-thesis-ba.pdf: error trying to connect: error:14090086:SSL routines:ssl3_get_server_certificate:certificate verify failed:s3_clnt.c:1264: (certificate has expired)
Caused By: There was an error while fetching "https://octarineparrot.com/assets/msci_paper.pdf", https://octarineparrot.com/assets/msci_paper.pdf: error trying to connect: error:14090086:SSL routines:ssl3_get_server_certificate:certificate verify failed:s3_clnt.c:1264: (certificate has expired)

@mark-i-m
Copy link
Member Author

The above also has the following:

2019-08-19T18:25:44.8638593Z error: process didn't exit successfully:/checkout/obj/build/x86_64-unknown-linux-gnu/stage2-tools/x86_64-unknown-linux-gnu/release/deps/client-4789b7d22e4fb0f9 (signal: 4, SIGILL: illegal instruction)

which is not comforting...

@ehuss
Copy link
Contributor

ehuss commented Aug 20, 2019

sigill is normal for a panic-while-panicking. The rls client tests tend to be a little flaky. cc @Xanewok, not sure if you've seen this error before. I would guess that the RlsHandle::drop is the cause of the second panic, though I can't really infer what caused the initial panic. I wish tests were run with RUST_BACKTRACE set.

@mark-i-m
Copy link
Member Author

Hopefully, should be fixed now.

@ehuss
Copy link
Contributor

ehuss commented Aug 21, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Aug 21, 2019

📌 Commit 9f14526 has been approved by ehuss

@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 Aug 21, 2019
@bors
Copy link
Contributor

bors commented Aug 21, 2019

⌛ Testing commit 9f14526 with merge 42dcd4b...

bors added a commit that referenced this pull request Aug 21, 2019
Update rustc-guide

Should fix toolstate failure

r? @ehuss
@bors
Copy link
Contributor

bors commented Aug 22, 2019

☀️ Test successful - checks-azure
Approved by: ehuss
Pushing 42dcd4b to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 22, 2019
@bors bors merged commit 9f14526 into rust-lang:master Aug 22, 2019
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #63705!

Tested on commit 42dcd4b.
Direct link to PR: #63705

🎉 rustc-guide on linux: test-fail → test-pass (cc @mark-i-m @spastorino, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Aug 22, 2019
Tested on commit rust-lang/rust@42dcd4b.
Direct link to PR: <rust-lang/rust#63705>

🎉 rustc-guide on linux: test-fail → test-pass (cc @mark-i-m @spastorino, @rust-lang/infra).
@mark-i-m mark-i-m deleted the fix-guide-1 branch December 11, 2019 21:35
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.

4 participants