Skip to content

Fix issues with MSVC and OneAPI CI due to VS 18 2026#6438

Open
jhendersonHDF wants to merge 4 commits into
HDFGroup:developfrom
jhendersonHDF:update_vs_version
Open

Fix issues with MSVC and OneAPI CI due to VS 18 2026#6438
jhendersonHDF wants to merge 4 commits into
HDFGroup:developfrom
jhendersonHDF:update_vs_version

Conversation

@jhendersonHDF

@jhendersonHDF jhendersonHDF commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Visual Studio 17 2022 seems to no longer be available from actions run on windows-latest from forks

Ref: actions/runner-images#14017

@jhendersonHDF jhendersonHDF requested a review from brtnfld June 10, 2026 14:21
@jhendersonHDF jhendersonHDF self-assigned this Jun 10, 2026
@jhendersonHDF jhendersonHDF requested a review from hyoklee as a code owner June 10, 2026 14:21
@jhendersonHDF jhendersonHDF added the Component - Testing Code in test or testpar directories, GitHub workflows label Jun 10, 2026
@github-project-automation github-project-automation Bot moved this to To be triaged in HDF5 - TRIAGE & TRACK Jun 10, 2026
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Checklist

This PR touches the following areas. Each needs at least one
sign-off from its listed owners before merging — an approval
covering only one area does not satisfy the others.


Intel_oneapi_windows:
name: "windows-oneapi ${{ inputs.build_mode }}"
runs-on: windows-latest

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The fortran-lang/setup-fortran action that's currently being used to setup OneAPI hasn't been updated yet for the latest windows, so this can only run on windows-2025 and not windows-latest (depending on where windows-latest gets mapped to, either windows-2025 or windows-2025-vs2026).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Correction, windows-2022 is needed currently until the OneAPI setup can be updated

@jhendersonHDF jhendersonHDF marked this pull request as draft June 10, 2026 15:24
@brtnfld brtnfld removed request for glennsong09 and lrknox June 10, 2026 15:49
glennsong09
glennsong09 previously approved these changes Jun 12, 2026
@jhendersonHDF jhendersonHDF force-pushed the update_vs_version branch 2 times, most recently from cd4bf69 to d3b125e Compare June 12, 2026 21:12
@brtnfld brtnfld removed the request for review from hyoklee June 12, 2026 21:12
@brtnfld brtnfld requested review from glennsong09 and removed request for glennsong09 June 12, 2026 21:30
force_java_implementation: "jni"
java_version: 11
win_runner: "windows-2022"
win_vs_generator: "Visual Studio 17 2022"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

CMake 3.26 doesn't support VS 18 2026, so this action has to use windows-2022 with VS 17 2022

- name: Setup Java
uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # v5.2.0
with:
java-version: 11

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

windows-2022 uses Java 8 by default whereas windows-latest uses 17. Since Java 11 is the minimum required, that has to be specified for windows-2022 runs

@jhendersonHDF jhendersonHDF marked this pull request as ready for review June 12, 2026 22:06
@jhendersonHDF jhendersonHDF changed the title Update Visual Studio versions in workflows Fix issues with MSVC and OneAPI CI due to VS 18 2026 Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component - Testing Code in test or testpar directories, GitHub workflows

Projects

Status: To be triaged

Development

Successfully merging this pull request may close these issues.

2 participants