-
Notifications
You must be signed in to change notification settings - Fork 788
[GHA] Uplift Linux IGC Dev RT version to igc-dev-3c5dac9 #15125
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
@jsji Any idea on this docker postcommit fail? https://github.com/intel/llvm/actions/runs/10561303761/job/29259192728
|
This is known limitation -- if the dev igc version we use (after uplift) are too old (more than 1 week old), we won't be able to download it from GitHub CI anymore, then it will fail. |
@jsji Thanks, I don't need to revert the PR right, this version just wont be used? Could we update the bot to close any PRs after the IGC max retention days passes? |
Yes, we don't. We will still use the old docker with very old version, until we uplift with new one succesfully.
We can do that, but it probably better if we could merge the PR when it is green, don't let it pass the date. |
Sure, I was out last week and it seems nobody else looks at these so I guess whoops :) |
Yeah, not your problem at all. Just wondering any other way that we can make sure we merge it on time? |
Make matrix/esimd teams codeowners for this file as they're the most interested in dev IGC testing. Ensure that they look into the failures in such PRs first, because I have an impression those are the tests that usually fail (might be wrong here). |
I think you're right and I talked with Jinsong offline and we're gonna do that, thanks |
PR fo codeowner update: #15193 |
Scheduled igc dev drivers uplift