Skip to content

L0 testing on BMG#21981

Open
rbanka1 wants to merge 3 commits into
intel:syclfrom
rbanka1:l0
Open

L0 testing on BMG#21981
rbanka1 wants to merge 3 commits into
intel:syclfrom
rbanka1:l0

Conversation

@rbanka1
Copy link
Copy Markdown
Contributor

@rbanka1 rbanka1 commented May 11, 2026

No description provided.

@rbanka1 rbanka1 requested a review from a team as a code owner May 11, 2026 14:48
@sarnex
Copy link
Copy Markdown
Contributor

sarnex commented May 11, 2026

Looks like the new CI is failing, please ping us when everything is fixed and this is ready for review, thanks!

Comment thread .github/workflows/ur-precommit.yml Outdated
- name: L0_BMG
runner: UR_L0_BMG
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
- name: L0_V2_BMG
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just this job should suffice, I don't think we need 4 more jobs

Comment thread .github/workflows/ur-precommit.yml Outdated
runner: UR_L0
other_adapter: NATIVE_CPU
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
- name: L0_BMG
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you cannot add _BMG to adapter's name - it's used to configure CMake: -DUR_BUILD_ADAPTER_${{matrix.adapter.name}}=ON.

Comment thread .github/workflows/ur-build-hw.yml Outdated
env:
ZE_ENABLE_LOADER_DEBUG_TRACE: 1
LIT_OPTS: "--timeout 120 -j 50"
LIT_OPTS: "--timeout 120 -j ${{ contains(inputs.adapter_name, 'BMG') && '4' || '50' }}"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you need to distinguish job count, you can check inputs.runner instead

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