-
Notifications
You must be signed in to change notification settings - Fork 734
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
[SYCL][DevOps] Fuse OCL CPU testing into GEN9 Linux task #9546
Conversation
05a631b
to
ca5bc3b
Compare
ca5bc3b
to
b145869
Compare
It would increase its duration by several minutes while eliminating 20+ minutes task completely.
b145869
to
a8cdcbf
Compare
"several minutes" - could you be more specific, please? I think it's worth noting that in your pre-commit the task took < 12 minutes (https://github.com/intel/llvm/actions/runs/5048597334/jobs/9057922308). |
33 vs 30 minutes
That was helpful. Turns out that we have As such, it would be even more helpful than I originally thought for the times with peak loads.
I think that by decreasing the load on AMD runners we will get more heavy build jobs dispatched onto them, freeing our GEN9 runners just for this fused task. |
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.
CI scripts changes look okay to me.
#9546 and similar accidentally broke code that was responsible to reset Intel GPU before running End-to-End tests. Restore functionality + add reset for the pre-commit task running End-to-End tests with a nightly build.
#9546 and similar accidentally broke code that was responsible to reset Intel GPU before running End-to-End tests. Restore functionality + add reset for the pre-commit task running End-to-End tests with a nightly build.
It would increase its duration by several minutes while eliminating 20+ minutes task completely.