Skip to content

[CI] additional docker volume mapping for E2E on Intel GPUs #15563

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

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

againull
Copy link
Contributor

@againull againull commented Oct 1, 2024

NEO runtime has env variables to emulate multiple root devices: NEOReadDebugKeys=1 CreateMultipleRootDevices=[num_devices] which seems to be useful to write E2E tests for multi-device scenario.

It turns out those env variables don't work under docker container in our CI because of missing system directory. Add additional volume mapping to fix this.

NEO runtime has env variables to emulate multiple root devices:
NEOReadDebugKeys=1 CreateMultipleRootDevices=[num_devices]
which seems to be useful to write E2E tests for multi-device scenario.

It turns out those env variables don't work under docker container in our CI
bacuse of missing system directory. Add additional volume mapping to
fix this.
@againull againull marked this pull request as ready for review October 1, 2024 04:16
@againull againull requested a review from a team as a code owner October 1, 2024 04:16
@againull
Copy link
Contributor Author

againull commented Oct 1, 2024

Failure on HIP is unrelated because this change doesn't affect it.

@againull againull merged commit 19516d0 into intel:sycl Oct 1, 2024
21 of 22 checks passed
sarnex pushed a commit that referenced this pull request Jan 8, 2025
`by-path` dir is required for `NEOReadDebugKeys=1
CreateMultipleRootDevices=[num_devices]`to work correctly.
Same as #15563
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.

2 participants