-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
ci: move x86_64-msvc-ext jobs to windows 2025 #136478
base: master
Are you sure you want to change the base?
Conversation
rustbot has assigned @Mark-Simulacrum. Use |
@bors try |
…, r=<try> ci: move x86_64-msvc-ext jobs to windows 2025 try-job: x86_64-msvc-ext1 try-job: x86_64-msvc-ext2 try-job: x86_64-msvc-ext3
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
weird error 🤔 |
The test is here: https://github.com/rust-lang/cargo/blob/0e3d73849ab8cbbab3ec5c65cbd555586cb21339/tests/testsuite/package.rs#L2847 On newer versions of Windows those "reserved" names are allowed so the test fails. Which Cargo seems to be aware of given the test is gated by https://github.com/rust-lang/cargo/blob/0e3d73849ab8cbbab3ec5c65cbd555586cb21339/crates/cargo-test-support/src/paths.rs#L373. However, that is set to always return |
Thanks a lot! Opened rust-lang/cargo#15135 |
<!-- Thanks for submitting a pull request 🎉! Here are some tips for you: * If this is your first contribution, read "Cargo Contribution Guide" first: https://doc.crates.io/contrib/ * Run `cargo fmt --all` to format your code changes. * Small commits and pull requests are always preferable and easy to review. * If your idea is large and needs feedback from the community, read how: https://doc.crates.io/contrib/process/#working-on-large-features * Cargo takes care of compatibility. Read our design principles: https://doc.crates.io/contrib/design.html * When changing help text of cargo commands, follow the steps to generate docs: https://github.com/rust-lang/cargo/tree/master/src/doc#building-the-man-pages * If your PR is not finished, set it as "draft" PR or add "WIP" in its title. * It's ok to use the CI resources to test your PR, but please don't abuse them. ### What does this PR try to resolve? Explain the motivation behind this change. A clear overview along with an in-depth explanation are helpful. You can use `Fixes #<issue number>` to associate this PR to an existing issue. ### How should we test and review this PR? Demonstrate how you test this change and guide reviewers through your PR. With a smooth review process, a pull request usually gets reviewed quicker. If you don't know how to write and run your tests, please read the guide: https://doc.crates.io/contrib/tests ### Additional information Other information you want to mention in this PR, such as prior arts, future extensions, an unresolved problem, or a TODO list. --> We are progressively moving the windows CI from windows 2022 to windows 2025 because we found windows 2025 more stable. In rust-lang/rust#136478 a cargo test failed and this might solve the issue. See [this](rust-lang/rust#136478 (comment)) comment. What do you think? If you have a better way of solving this, let me know 👍
The cargo PR was merged. Now we need to wait for the cargo team to update cargo inside this repository. |
352f5ab
to
131e3fe
Compare
@bors try |
…, r=<try> ci: move x86_64-msvc-ext jobs to windows 2025 try-job: x86_64-msvc-ext1 try-job: x86_64-msvc-ext2 try-job: x86_64-msvc-ext3
☀️ Try build successful - checks-actions |
@rustbot ready |
IIUC, this continues to decrease our test coverage for older Windows, right? Arguably it makes Windows 2025 "more" a tier 1 target and Windows 10(?) more of a tier 2 target. Historically we haven't made much of that, but I'm wondering if folks more familiar with Windows think we should be trying to do something else here or at least documenting that change somewhere in our platform support policies, presuming I'm following correctly. |
I'd note for completeness that "Windows 10" isn't a single OS version. It spans about 7 years of development and 12 "feature" releases (e.g. new APIs, etc). Even with Windows Server 2019 we're testing way above RTM Windows 10. Arguably for our use the change from Server 2016 to Server 2019 was a much bigger jump than from 2019 to 2025 because the newer APIs we conditionally use are all available on 2019. Which is to say this has long been an issue but, yeah, it's mostly a documentation issue given our constraints. Ultimately it would be great if we have more versions to test in CI so perhaps we can make more use of the ones we do have access to. Maybe we could have a job that runs |
Yeah, makes sense. I'd be OK not doing anything in this PR, I think, given the current state of things, but it would probably be good to have some discussion/updates to the platform support docs to reflect the reality of what we test (and don't test).
It sounds plausible so long as we have relatively easy access to older Windows, which I guess probably isn't going to be true for long? Not sure what the plans are in Microsoft or GitHub for long-term availability of the Windows 10 runners (or Windows 10 itself). I'm OK moving forward with this if there's no blockers from the windows maintainers side (r=me). |
These jobs are a bit flaky. See this zulip thread and this GitHub issue.
Windows 2025 is working well for
x86_64-msvc
jobs since we moved them in #135632Analysis
try-job: x86_64-msvc-ext1
try-job: x86_64-msvc-ext2
try-job: x86_64-msvc-ext3