Skip to content

Update pinned oldest win image on azure #29992

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented May 1, 2025

The windows-2019 image will be deprecated on 1.6.2025 (#29797 (comment)). Also, this is a test whether the timeouts #29797 are related to the windows-2019 image.

The windows-2019 image will be deprecated on 1.6.2025.
Also, this is a test whether the timeouts matplotlib#29797 are related to the
windows-2019 image.
@timhoffm timhoffm force-pushed the update-winimage branch from ce60580 to aea28b6 Compare May 1, 2025 07:21
@timhoffm
Copy link
Member Author

timhoffm commented May 1, 2025

First run did not exhibit timeout. Force push to rerun CI. Let's check whether this is reliable by running CI multiple times.

@timhoffm
Copy link
Member Author

timhoffm commented May 1, 2025

Second run did not exhibit timeout. Force push to rerun CI.

@timhoffm
Copy link
Member Author

timhoffm commented May 1, 2025

This did not have timeouts three times in a row.

I suggest to merge this and see whether our timouts are gone. We would have to update the image in a month anyway due to the deprecation, and I don't think it's a relevant miss if we don't run the windows-2019 image.

@timhoffm timhoffm marked this pull request as draft May 1, 2025 08:39
@timhoffm timhoffm marked this pull request as ready for review May 1, 2025 08:40
@rcomer
Copy link
Member

rcomer commented May 1, 2025

There is an if-loop at line 123 that the comment states is for windows-2019. I guess that can be removed.

The windows-2019 image will be deprecated on 1.6.2025.
Also, this is a test whether the timeouts matplotlib#29797 are related to the
windows-2019 image.
@timhoffm timhoffm force-pushed the update-winimage branch from 4958941 to 3458e53 Compare May 2, 2025 06:32
@@ -100,25 +100,6 @@ stages:
echo "Coverage session ID: ${SESSION_ID}"
VS=$(ls -d /c/Program\ Files*/Microsoft\ Visual\ Studio/*/Enterprise)
echo "Visual Studio: ${VS}"
DIR="$VS/Common7/IDE/Extensions/Microsoft/CodeCoverage.Console"
if [[ -d $DIR ]]; then
# This is for MSVC 2022 (on windows-latest).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be the opposite of what you wanted to do, this comment indicates this is for 2022, not 2019. (And the tests failed without actually running, so that tracks)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants