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

Align dates in rustup toolchains with those in rustc -V #51764

Closed
SimonSapin opened this issue Jun 24, 2018 · 3 comments
Closed

Align dates in rustup toolchains with those in rustc -V #51764

SimonSapin opened this issue Jun 24, 2018 · 3 comments
Labels
T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@SimonSapin
Copy link
Contributor

The previous attempt rust-lang/rust-central-station#27 was reverted because things broke. This probably needs someone from @rust-lang/infra to do this more carefully:

Original description:


rustup update tells me that the current Nightly is rustc 1.24.0-nightly (cfba0d446 2017-12-05). This is the output of rustc --version. It’s tempting to think that I can pin to this version by specifying a nightly-2017-12-05 toolchain, but that actually refers to yesterday’s Nightly, rust version 1.24.0-nightly (8503b3ff8 2017-12-04). This discrepancy also makes statements like “I can reproduce bug X in Nightly 2017-12-05” ambiguous.

The reason is that rustc -V contains the commit date of whatever happened to be the latest commit in master when a Nightly was published, but the archive URL (and so the toolchain names) contains the date that is current at that time. Since publication is started at midnight UTC, the latest commit is almost always "yesterday" and the dates end up off by one.

(This is technically a race condition but one that commits can never win since testing a PR takes a couple hours. Bors makes a merge commit before testing, and (if successful) pushes to master afterwards. So the latest commit in master is always at least a couple hours old.)

[This] PR changes the archive URL / toolchain name date to that of "10 minutes ago" instead of "now", soon after Cron starts this program at midnight UTC. That way, the date will most often match that the latest commit. (It can still be off if nothing was merged the day before, but given PR traffic on rust-lang/rust that’s relatively rare.)

@SimonSapin SimonSapin added the T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. label Jun 24, 2018
@SimonSapin
Copy link
Contributor Author

(Perhaps this issue belongs in https:/rust-lang/rust-central-station, but I don’t know if it’s followed as much and I can’t set labels there.)

@Mark-Simulacrum
Copy link
Member

This does seem like a good place to have this. I think the first thing to fix here is rust-lang/rust-central-station#50 which should actually be fairly easy to do if someone's interested -- see the notes in rust-lang/rust-central-station#50 (comment); I'd be interested in an evaluation of how much code moving the manifest construction into central station would "drag with it" from x.py. My hope is the answer is "not much".

@Mark-Simulacrum
Copy link
Member

Closing as duplicate of #51533.

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

No branches or pull requests

2 participants