Skip to content

Conversation

bb-sycl
Copy link
Contributor

@bb-sycl bb-sycl commented Sep 23, 2025

Scheduled drivers uplift

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex force-pushed the ci/update_gpu_driver-linux-25.35.35096.9 branch from dbf11eb to 18fd16f Compare September 24, 2025 19:15
@sarnex sarnex marked this pull request as ready for review September 25, 2025 14:21
@sarnex sarnex requested review from a team as code owners September 25, 2025 14:21
Copy link
Contributor

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@sarnex
Copy link
Contributor

sarnex commented Sep 25, 2025

@aelovikov-intel @uditagarwal97 Can you guys review the CI change here? Thx

uses: ./devops/actions/reset_gpu
- name: Install drivers
if: inputs.install_igc_driver == 'true' || inputs.install_dev_igc_driver == 'true'
if: inputs.e2e_binaries_artifact != 'in-container' && (inputs.install_igc_driver == 'true' || inputs.install_dev_igc_driver == 'true')
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we do something like image.contains("sycl-rel-") instead? I think that would be more direct.

Copy link
Contributor

Choose a reason for hiding this comment

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

I considered that, but I thought that having a rule that any containers with E2E binaries inside them require some kind of stable environment, so the GPU driver should be stable as well.
If you don't think that rule is necessarily true I can make this change.

@sarnex sarnex merged commit 41604ef into sycl Sep 25, 2025
58 of 62 checks passed
@bader bader deleted the ci/update_gpu_driver-linux-25.35.35096.9 branch September 29, 2025 04:50
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.

5 participants