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

rustup can't update nightly #51699

Closed
durka opened this issue Jun 22, 2018 · 20 comments
Closed

rustup can't update nightly #51699

durka opened this issue Jun 22, 2018 · 20 comments
Labels
regression-from-stable-to-nightly Performance or correctness regression from stable to nightly. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue.

Comments

@durka
Copy link
Contributor

durka commented Jun 22, 2018

There seems to be a malformed URL in the rustup manifest since 1.27 got released today.

durska:rust alex$ rustup self update
info: checking for self-updates
durska:rust alex$ rustup update
info: syncing channel updates for 'stable-x86_64-apple-darwin'
info: syncing channel updates for 'beta-x86_64-apple-darwin'
info: syncing channel updates for 'nightly-x86_64-apple-darwin'
info: latest update on 2018-06-22, rust version 1.28.0-nightly (662c70a59 2018-06-21)
info: downloading component 'rustc'
error: failed to parse url: 1.28.0-nightly (662c70a59 2018-06-21)/2018-06-22/rustc-nightly-x86_64-apple-darwin.tar.xz
info: checking for self-updates

       stable-x86_64-apple-darwin unchanged - rustc 1.27.0 (3eda71b00 2018-06-19)
         beta-x86_64-apple-darwin unchanged - rustc 1.28.0-beta.1 (8e6052e77 2018-06-21)
  nightly-x86_64-apple-darwin update failed - rustc 1.28.0-nightly (990d8aa74 2018-05-25)

durska:rust alex$
@ishitatsuyuki
Copy link
Contributor

Related to #51680?

@nrc
Copy link
Member

nrc commented Jun 22, 2018

This is causing problems on Travis too, see for example rustfmt

@nrc nrc added I-nominated T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Jun 22, 2018
@Havvy
Copy link
Contributor

Havvy commented Jun 22, 2018

This affects the mozilla overlay for nixpkgs as well, since that uses the same URLs.

@mati865
Copy link
Contributor

mati865 commented Jun 22, 2018

Comparison of nightly and stable metadata for rustup:

# channel-rust-nightly.toml
[pkg.cargo.target.x86_64-unknown-linux-gnu]
available = true
url = "1.28.0-nightly (662c70a59 2018-06-21)/2018-06-22/cargo-nightly-x86_64-unknown-linux-gnu.tar.gz"
hash = "039fc11d15fab9951061ee25991708659f6a3900d58fcd0a3db9e6f8b4987d51"
xz_url = "1.28.0-nightly (662c70a59 2018-06-21)/2018-06-22/cargo-nightly-x86_64-unknown-linux-gnu.tar.xz"
xz_hash = "27613570b55d12e7f3b66cf5296a41c8d7a8ee1c95cce7f9cfedee3f201b1e11"

# channel-rust-stable.toml
[pkg.cargo.target.x86_64-unknown-linux-gnu]
available = true
url = "https://static.rust-lang.org/dist/2018-06-21/cargo-0.28.0-x86_64-unknown-linux-gnu.tar.gz"
hash = "3612ae45bf4504836810a74e6cef177358fce7219794013536c354bbd6ee61ef"
xz_url = "https://static.rust-lang.org/dist/2018-06-21/cargo-0.28.0-x86_64-unknown-linux-gnu.tar.xz"
xz_hash = "386ba6e3c0a2b1718fe0487d03f96e1ad12796cfb44813dfc22c5be54657e7a4"

Somehow https://static.rust-lang.org/dist got replaced by 1.28.0-nightly (662c70a59 2018-06-21) for latest nightly, other things seem fine.
Opening https://static.rust-lang.org/dist/2018-06-22/cargo-nightly-x86_64-unknown-linux-gnu.tar.gz in the browser starts downloading of the archive.

@kennytm kennytm added the regression-from-stable-to-nightly Performance or correctness regression from stable to nightly. label Jun 22, 2018
@kennytm
Copy link
Member

kennytm commented Jun 22, 2018

Regression caused by #50336.

diff --git a/src/bootstrap/dist.rs b/src/bootstrap/dist.rs
index 9092deeac461..73cda4b436d4 100644
--- a/src/bootstrap/dist.rs
+++ b/src/bootstrap/dist.rs
@@ -1756,6 +1740,7 @@ impl Step for HashSign {
         cmd.arg(builder.package_vers(&builder.release_num("cargo")));
         cmd.arg(builder.package_vers(&builder.release_num("rls")));
         cmd.arg(builder.package_vers(&builder.release_num("rustfmt")));
+        cmd.arg(builder.llvm_tools_vers());
         cmd.arg(addr);
 
         builder.create_dir(&distdir(builder));

@mati865
Copy link
Contributor

mati865 commented Jun 22, 2018

@kennytm you were few minutes faster.
build-manifest tool doesn't expect additional argument:

let rls_release = args.next().unwrap();
let s3_address = args.next().unwrap();

Btw rustfmt arg isn't there as well 😕 Forget about it, old commit

@kennytm
Copy link
Member

kennytm commented Jun 22, 2018

@mati865 You're looking at an old commit 😄

let rls_release = args.next().unwrap();
let rustfmt_release = args.next().unwrap();
let s3_address = args.next().unwrap();

@mati865
Copy link
Contributor

mati865 commented Jun 22, 2018

@kennytm Oops, commit from 27 Jul 2017.
Probably I should've had a bit more sleep 😒

@wesbz
Copy link

wesbz commented Jun 22, 2018

Updating with a previous version such as rustup update nightly-2018-06-18 seems to work for me...

@RalfJung
Copy link
Member

This is causing problems on Travis too

Indeed, this breaks every single project having CI against Rust nightly.

mykmelez added a commit to mykmelez/rkv that referenced this issue Jun 22, 2018
mykmelez added a commit to mozilla/rkv that referenced this issue Jun 22, 2018
rust-lang/rust#51699 has broken the nightly Rust toolchain. Our Travis configuration allows failures when building with that toolchain, but we use nightly rustfmt when building with all toolchains, so the bustage affects our stable and beta builds too.

To work around the problem, this branch uses a specific rustfmt nightly rather than the latest one. (We might want to keep doing this even after the bustage is fixed, to avoid such bustage in the future.)
@est31
Copy link
Member

est31 commented Jun 22, 2018

For anyone wondering, PR #51703 has fixed the bug. It will take until the next scheduled nightly upload until rustup starts working again.

@durka
Copy link
Contributor Author

durka commented Jun 22, 2018

Should rustup tests be added as part of the release process?

@pietroalbini
Copy link
Member

There is already an open issue about testing rustup. rust-lang/rust-central-station#17

@slaiyer
Copy link

slaiyer commented Jun 23, 2018

@est31 when is the next scheduled nightly upload?

@SimonSapin
Copy link
Contributor

@slaiyer It should be in progress right now. https://github.com/rust-lang/rust-central-station/blob/12b28bc1c2/crontab#L7 runs at midnight UTC.

@ZhangHanDong
Copy link

✗ rustup update nightly
info: syncing channel updates for 'nightly-x86_64-apple-darwin'
info: latest update on 2018-06-22, rust version 1.28.0-nightly (662c70a59 2018-06-21)
info: downloading component 'rustc'
error: failed to parse url: 1.28.0-nightly (662c70a59 2018-06-21)/2018-06-22/rustc-nightly-x86_64-apple-darwin.tar.xz
info: caused by: relative URL without a base

@Mark-Simulacrum
Copy link
Member

Nightly should release within the next 30 minutes, probably the next 10-15 minutes. Until then, yes, the command won't work.

@kazcw
Copy link

kazcw commented Jun 23, 2018

2018-06-23 is live and the url is fixed

@grossws
Copy link

grossws commented Jan 10, 2019

Same issue with current meta for nightly:

error: failed to parse url: nightly/2019-01-10/rustc-nightly-x86_64-unknown-linux-gnu.tar.xz
info: caused by: relative URL without a base

From https://static.rust-lang.org/dist/channel-rust-nightly.toml:

manifest-version = "2"
date = "2019-01-10"

[pkg.rustc.target.x86_64-unknown-linux-gnu]
available = true
url = "nightly/2019-01-10/rustc-nightly-x86_64-unknown-linux-gnu.tar.gz"
hash = "f4b5c77ce75ffdee97d460f03b8f84dd49129143c8e896e44d4512de2c0a686a"
xz_url = "nightly/2019-01-10/rustc-nightly-x86_64-unknown-linux-gnu.tar.xz"
xz_hash = "54a1b1791612651ded4c052c2c29a9b2e681344da4cebdc2382d14b62936f4fa"

@kennytm
Copy link
Member

kennytm commented Jan 10, 2019

@grossws Please reply to the stickied issue #57488, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-from-stable-to-nightly Performance or correctness regression from stable to nightly. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests