-
Notifications
You must be signed in to change notification settings - Fork 787
[CI] Refactor sycl-containers to use matrix #13292
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
Conversation
@intel/llvm-gatekeepers Please help to merge . Thanks. |
build_args: "use_latest=false" | ||
- name: Intel Drivers (unstable) Ubuntu 22.04 Docker image | ||
file: ubuntu2204_intel_drivers | ||
tags: ubuntu2204_intel_drivers_unstable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this patch changes tags we used for uploading docker images.
@jsji, could you update the documentation https://github.com/intel/llvm/blob/sycl/sycl/doc/developer/DockerBKMs.md#sycl-containers-overview if it's needed, please?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the tag for unstable
is changed, but we never mentioned/use it anywhere in source tree/docs. So I don't think any updated needed. Let me know if there is some usage that I don't know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will update the doc to describe how to test with dev igc later when the whole workflow is enabled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#13307 for the image update first.
Address review comments from #13272 (comment)