Skip to content

ci: Switch to VS 2026 for windows-2025 image - #31497

Closed
mbroz wants to merge 1 commit into
openssl:masterfrom
mbroz:test
Closed

mbroz wants to merge 1 commit into
openssl:masterfrom
mbroz:test

Conversation

@mbroz

@mbroz mbroz commented Jun 14, 2026

Copy link
Copy Markdown
Member

Current Windows CI builds are failing due to migration described in actions/runner-images#14017

GitHub no longer supports Visual Studio 2022 on windows-2025 image, switch to VS 2026.

An alternative is to use windows-2022 image, but then the test matrix does no longer make sense (the goal was to test new Windows here).

GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.
@mbroz
mbroz requested a review from quarckster as a code owner June 14, 2026 18:14
@mbroz mbroz added branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 branch: 4.0 Applies to openssl-4.0 labels Jun 14, 2026
vcvars: C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvars64.bat
- os: windows-2025
vcvars: C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvars64.bat
vcvars: C:\Program Files\Microsoft Visual Studio\18\Enterprise\VC\Auxiliary\Build\vcvars64.bat

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there some way to use %VSINSTALLDIR% here?

@mbroz mbroz Jun 15, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Well, there could be multiple VS versions installed (and even versions like community & enterprise)
I am not sure it is reliable. The path above is the default intallation path that should work.

Anyway, that is for @quarckster and a different patch, this one just unbreaks the build :)

@openssl-machine openssl-machine added the approval: review pending This pull request needs review by a committer label Jun 15, 2026
@quarckster

Copy link
Copy Markdown
Member

LGTM

@mbroz
mbroz requested a review from t8m June 15, 2026 07:13
@t8m t8m added severity: urgent Fixes an urgent issue (exempt from 24h grace period) tests: exempted The PR is exempt from requirements for testing labels Jun 15, 2026

@t8m t8m left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is urgent as Windows CI is broken.

@t8m
t8m requested a review from slontis June 15, 2026 07:20
@openssl-machine openssl-machine added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Jun 15, 2026
openssl-machine pushed a commit that referenced this pull request Jun 15, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from #31497)

(cherry picked from commit 164c156)
openssl-machine pushed a commit that referenced this pull request Jun 15, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from #31497)

(cherry picked from commit 164c156)
openssl-machine pushed a commit that referenced this pull request Jun 15, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from #31497)
openssl-machine pushed a commit that referenced this pull request Jun 15, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from #31497)

(cherry picked from commit 164c156)
openssl-machine pushed a commit that referenced this pull request Jun 15, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from #31497)

(cherry picked from commit 164c156)
@t8m

t8m commented Jun 15, 2026

Copy link
Copy Markdown
Member

Merged to the master, 4.0, 3.6, 3.5 and 3.4 branches. Thank you.

@t8m t8m closed this Jun 15, 2026

@esyr esyr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Concur urgent.

@esyr esyr mentioned this pull request Jun 15, 2026
2 tasks
n13l pushed a commit to n13l/openssl that referenced this pull request Jul 25, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from openssl#31497)
rickyringler pushed a commit to rickyringler/openssl that referenced this pull request Aug 21, 2026
GitHub no longer supports Visual Studio 2022 on windows-2025
image, switch to VS 2026.

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jun 15 07:23:17 2026
(Merged from openssl#31497)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 branch: 4.0 Applies to openssl-4.0 severity: urgent Fixes an urgent issue (exempt from 24h grace period) tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants