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

nll crater run for 1.40 #65897

Closed
Mark-Simulacrum opened this issue Oct 28, 2019 · 681 comments
Closed

nll crater run for 1.40 #65897

Mark-Simulacrum opened this issue Oct 28, 2019 · 681 comments
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Comments

@Mark-Simulacrum
Copy link
Member

This is to hopefully give a more up to date impression of regressions for the upcoming NLL post.

@Mark-Simulacrum
Copy link
Member Author

@craterbot run start=beta-2019-10-27 end=nightly-2019-10-27 mode=check-only cap-lints=allow p=1

@craterbot
Copy link
Collaborator

👌 Experiment pr-65897 created and queued.
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added the S-waiting-on-crater Status: Waiting on a crater run to be completed. label Oct 28, 2019
@craterbot
Copy link
Collaborator

🚧 Experiment pr-65897 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@Mark-Simulacrum
Copy link
Member Author

@craterbot retry-report

@craterbot
Copy link
Collaborator

🚨 Error: generation of the report of the pr-65897 experiment didn't fail!

🆘 If you have any trouble with Crater please ping @rust-lang/infra!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@Aaron1011
Copy link
Member

Crater seems to be stuck

@Mark-Simulacrum
Copy link
Member Author

Maybe try this again, not sure... @pietroalbini probably needs to take a look.

@craterbot retry-report

@craterbot
Copy link
Collaborator

🚨 Error: generation of the report of the pr-65897 experiment didn't fail!

🆘 If you have any trouble with Crater please ping @rust-lang/infra!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@pietroalbini
Copy link
Member

@craterbot retry-report

@craterbot
Copy link
Collaborator

🛠️ Generation of the report for pr-65897 queued again.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🎉 Experiment pr-65897 is completed!
📊 1112 regressed and 3460 fixed (77633 total)
📰 Open the full report.

⚠️ If you notice any spurious failure please add them to the blacklist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Nov 4, 2019
@jonas-schievink
Copy link
Contributor

That's a lot of borrowck regressions. cc @matthewjasper

@matthewjasper
Copy link
Contributor

matthewjasper commented Nov 4, 2019

There are also a lot of regressions due to pre expansion gating and fixes because features are gated on beta but not nightly. Could another run be done on the regressed crates with the current nightly?

@Mark-Simulacrum
Copy link
Member Author

Oh, yeah, to be clear I had not intended for anyone to take a look yet, this was a preliminary run that we can definitely reduce down somewhat :)

@craterbot run start=beta-2019-11-02 end=try#5e633973aa9255a88b65a39484627bb54e6044b3 mode=check-only cap-lints=allow p=1 crates=https://gist.githubusercontent.com/Mark-Simulacrum/b2b7233c91ea255e3d6d053b1c2bd821/raw/c7f32c4fe19446e3e08f44b82708c3db9d0ceea2/nll-crates.txt

This should be a much reduced run which compares just these crates with a more recent commit on the end point which is in fact compiled as if it was beta (avoiding unstable feature related regressions/fixes, hopefully -- this commit is stolen from #66063)

@craterbot
Copy link
Collaborator

👌 Experiment pr-65897-1 created and queued.
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 4, 2019
@craterbot
Copy link
Collaborator

🚧 Experiment pr-65897-1 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

8 similar comments
@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

12 similar comments
@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🚨 Experiment pr-65897-1 has encountered an error: command "/workspace/cargo-home/bin/rustup" "toolchain" "install" "beta-2019-11-02" failed
🛠️ If the error is fixed use the retry command.

🆘 Can someone from the infra team check in on this? @rust-lang/infra
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@TimNN
Copy link
Contributor

TimNN commented Nov 4, 2019

@craterbot abort

@craterbot
Copy link
Collaborator

🗑️ Experiment pr-65897-1 deleted!

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Nov 4, 2019
@rust-lang rust-lang unlocked this conversation Nov 4, 2019
@Mark-Simulacrum
Copy link
Member Author

@craterbot run start=beta-2019-11-03 end=try#5e633973aa9255a88b65a39484627bb54e6044b3 mode=check-only cap-lints=allow p=1 crates=https://gist.githubusercontent.com/Mark-Simulacrum/b2b7233c91ea255e3d6d053b1c2bd821/raw/c7f32c4fe19446e3e08f44b82708c3db9d0ceea2/nll-crates.txt

(per https://static.rust-lang.org/dist/2019-11-03 this date should be correct -- off by one errors, I tell you!)

@craterbot
Copy link
Collaborator

👌 Experiment pr-65897-1 created and queued.
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 4, 2019
@Aaron1011
Copy link
Member

Something gives me the feeling that Experiment pr-65897-1 has encountered an error 😄

@craterbot
Copy link
Collaborator

🚧 Experiment pr-65897-1 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🎉 Experiment pr-65897-1 is completed!
📊 1096 regressed and 0 fixed (1112 total)
📰 Open the full report.

⚠️ If you notice any spurious failure please add them to the blacklist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Nov 6, 2019
@Mark-Simulacrum
Copy link
Member Author

The majority of these are known -- in particular, nalgebra, rusttype and url still have a lot of reverse-dependencies that haven't updated on github (and that have lockfiles). I think those aren't too interesting, we can exclude them for the most part -- that set needs to just cargo update to resolve the problem to our knowledge.

There are at least 53 (around 60 total, I believe) crates.io crates that break due to the NLL switch -- I think we should try to get patches land in all of those over the next 6 weeks so that by the time this hits stable we hopefully only have regressions in a few or no crates.io crates.

See https://hackmd.io/@simulacrum/BkkP1gbor for a list of roots from this run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

No branches or pull requests

7 participants