-
Notifications
You must be signed in to change notification settings - Fork 769
[CI] Enable v2 adapter testing on BMG #18357
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
base: sycl
Are you sure you want to change the base?
Conversation
9f63093
to
6f011d1
Compare
6f011d1
to
fc25582
Compare
fc25582
to
ec8a3f4
Compare
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.
@sarnex , @uditagarwal97 , do we have enough runners capacity in CI for that?
Here's the time taken by E2E jobs before and after this PR: Overhead on PVC seems significant to me. IIRC, we also had one PVC runner also used for build. @sarnex have you seen PVC being a bottleneck recently? I think we should be good with BMG, although we have only one Linux BMG runner, I haven't experienced BMG being a bottleneck recently. |
PVC GPU hasn't been a blocker recently, but I agree the more than doubling of the job time is worrying. |
That seems to be due to the timeout, so not representative. That said, timeout in itself is a problem. |
Ah sorry, it would be nice to see some results we think are representative of normal runs. |
Yes, I'm investigating the timeouts right now - it seems to be related to copy offload (so likely a driver issue). For now, perhaps we can enable the testing just for BMG? |
👍 |
No description provided.