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

[rust] update 1.44.1, code coverage tested as well -- adding in backtraces #4708

Closed
wants to merge 1 commit into from
Closed

[rust] update 1.44.1, code coverage tested as well -- adding in backtraces #4708

wants to merge 1 commit into from

Conversation

rexhoffman
Copy link
Contributor

Motivation

Rust update with CI tests.

Have you read the [Contributing Guidelines on pull requests]

y

Test Plan

CI, plus temporary coverage report.

Related PRs

none

@codecov-commenter
Copy link

Codecov Report

Merging #4708 into master will decrease coverage by 0.2%.
The diff coverage is 34.8%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #4708     +/-   ##
========================================
- Coverage    39.8%   39.6%   -0.3%     
========================================
  Files         835     826      -9     
  Lines      100719   97522   -3197     
  Branches    47433   45583   -1850     
========================================
- Hits        40157   38663   -1494     
- Misses      22044   22555    +511     
+ Partials    38518   36304   -2214     
Impacted Files Coverage Δ
config/management/src/key.rs 40.9% <0.0%> (+4.1%) ⬆️
config/management/src/layout.rs 26.1% <0.0%> (-0.5%) ⬇️
config/management/src/smoke_test.rs 1.3% <0.0%> (-0.1%) ⬇️
config/management/src/storage_helper.rs 45.5% <0.0%> (-0.5%) ⬇️
config/management/src/verify.rs 20.4% <0.0%> (+0.6%) ⬆️
config/management/src/waypoint.rs 0.0% <0.0%> (ø)
config/src/config/mod.rs 50.5% <0.0%> (-0.6%) ⬇️
config/src/config/safety_rules_config.rs 63.1% <0.0%> (-3.6%) ⬇️
consensus/consensus-types/src/block_retrieval.rs 39.3% <0.0%> (ø)
consensus/consensus-types/src/block_test.rs 19.5% <0.0%> (ø)
... and 205 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb586d4...7730529. Read the comment docs.

@rexhoffman rexhoffman changed the title attempt rust 1.44.1 [rust] update 1.44.1, code coverage tests as well Jun 24, 2020
@rexhoffman rexhoffman changed the title [rust] update 1.44.1, code coverage tests as well [rust] update 1.44.1, code coverage tested as well -- adding in backtraces Jun 24, 2020
@rexhoffman rexhoffman marked this pull request as ready for review June 24, 2020 20:05
@metajack
Copy link
Contributor

@bors-libra r+ p=-1

@bors-libra
Copy link
Contributor

📌 Commit 8b382d9 has been approved by metajack

@bors-libra
Copy link
Contributor

⌛ Testing commit 8b382d9 with merge 2056bdd...

@github-actions
Copy link

Cluster Test Result

all up : 1293 TPS, 3491 ms latency, 4250 ms p99 latency, no expired txns

Repro cmd:

./scripts/cti --tag land_2056bddc --run bench

@bors-libra
Copy link
Contributor

☀️ Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing 2056bdd to master...

@agc-quadra
Copy link

This PR appears to have broken rust on Mac:

libra/ % cargo build
error: process didn't exit successfully: rustc -vV (exit code: 1)
--- stderr
error: 'rustc' is not installed for the toolchain '1.44.1-x86_64-apple-darwin'
To install, run rustup component add rustc --toolchain 1.44.1-x86_64-apple-darwin

libra/ % rustup component add rustc --toolchain 1.44.1-x86_64-apple-darwin
error: toolchain '1.44.1-x86_64-apple-darwin' does not support components
libra/ %

@rexhoffman

@metajack
Copy link
Contributor

metajack commented Jun 25, 2020

This PR appears to have broken rust on Mac:

Your toolchain has become corrupted. This happens when your IDE triggers a toolchain update and another one starts at the same time.

To get back to a working setup, just uninstall and reinstall the toolchain.

rustup toolchain uninstall 1.44.1
rustup toolchain install 1.44.1

@agc-quadra
Copy link

Didn't work. The only way I can get to a working setup (so far) is to go to a branch before this PR.

error: 'rustc' is not installed for the toolchain '1.44.0-x86_64-apple-darwin'
To install, run `rustup component add rustc --toolchain 1.44.0-x86_64-apple-darwin`

libra % rustup toolchain uninstall 1.44.1

info: uninstalling toolchain '1.44.1-x86_64-apple-darwin'
info: toolchain '1.44.1-x86_64-apple-darwin' uninstalled
libra % rustup toolchain install 1.44.1

info: syncing channel updates for '1.44.1-x86_64-apple-darwin'
471.9 KiB / 471.9 KiB (100 %) 415.0 KiB/s in  1s ETA:  0s
info: latest update on 2020-06-18, rust version 1.44.1 (c7087fe00 2020-06-17)
info: downloading component 'cargo'
  3.9 MiB /   3.9 MiB (100 %) 1023.3 KiB/s in  2s ETA:  0s
info: downloading component 'clippy'
info: downloading component 'rust-docs'
 12.2 MiB /  12.2 MiB (100 %)  10.3 MiB/s in  1s ETA:  0s
info: downloading component 'rust-std'
 16.6 MiB /  16.6 MiB (100 %)   9.2 MiB/s in  1s ETA:  0s
info: downloading component 'rustc'
 56.1 MiB /  56.1 MiB (100 %)  10.4 MiB/s in  5s ETA:  0s
info: downloading component 'rustfmt'
info: installing component 'cargo'
info: installing component 'clippy'
info: installing component 'rust-docs'
 12.2 MiB /  12.2 MiB (100 %)   8.5 MiB/s in  1s ETA:  0s
info: installing component 'rust-std'
info: installing component 'rustc'
 56.1 MiB /  56.1 MiB (100 %)  17.6 MiB/s in  3s ETA:  0s
info: installing component 'rustfmt'

  1.44.1-x86_64-apple-darwin installed - rustc 1.44.1 (c7087fe00 2020-06-17)

info: checking for self-updates
 libra % cargo build
error: process didn't exit successfully: `rustc -vV` (exit code: 1)
--- stderr
error: 'rustc' is not installed for the toolchain '1.44.0-x86_64-apple-darwin'
To install, run `rustup component add rustc --toolchain 1.44.0-x86_64-apple-darwin`

I've attempted to completely uninstall rust with rustup self uninstall and then do a clean install, only to recreate the same effect.

@metajack
Copy link
Contributor

The error messages you are posting are requesting 1.44.0 and you're installing 1.44.1. Do you have an override set? What does your rust-toolchain file say when you are on master?

@agc-quadra
Copy link

The error messages you are posting are requesting 1.44.0 and you're installing 1.44.1. Do you have an override set? What does your rust-toolchain file say when you are on master?

Very good questions. I came back and did a fetch / rebase on master. And then rustup self uninstall; at which point rustup self install no longer worked (?). So I used the scripts/dev_setup.sh and everything seems to be happy again. Of course I tried repeating the uninstall/install suggestion before looking for the rust-toolchain file, so I can't tell you what state it was in.

Thanks a bunch for the patience. I'm glad the solution can be explained by my local incompetence.

One thought on where everything went sideways --- I keep cargo watch -x build running in a terminal; it's possible that that process interfered with the toolchain update on master and borked the world.

@metajack
Copy link
Contributor

rustup does not lock it's installation during install, so if anything triggers an install and quits before continuing or another rustup also attemps to install and fails, it leaves the directory in a bad state. cargo watch would definitely be suspect here, as is most Rust IDE tooling that uses rust-analyzer or RLS, since those all invoke rustup under the hood to perform some operations.

It's a known limitation of the tooling in rustup: rust-lang/rustup#988

@rexhoffman rexhoffman deleted the rust_1_44_1 branch July 6, 2020 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants