-
Notifications
You must be signed in to change notification settings - Fork 10.6k
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
nix: ci: fit into the new limits #6346
Comments
Until ggerganov#6346 is resolved
Until ggerganov#6346 is resolved
Hi @mscheong01! Please advice, could we afford more lax limits for
Even if we implement e.g. ccache, it would be nice to run the pure builds once in a while |
this could be done by checking the
I can't think of a straightforward solution that wouldn't unnecessarily complicate the workflow settings 🤔. One way would be to add a manual trigger that doesn't get cancelled by other runs, but I don't think it's the best option. |
HIP is probably is the most important build for nix, since it is not tested in any other workflow, as far as I know. Most of the other nix builds are redundant. |
Until ggerganov#6346 is resolved
Until ggerganov#6346 is resolved
This issue was closed because it has been inactive for 14 days since being marked as stale. |
Most (all) of the nix-build jobs are being cancelled in progress since the quotas have changed. Adjust the workflows to fit in the new limits.
Context: since #6243 the ci jobs are grouped by refs and cancelled together. The existing "Nix CI" job wasn't prepared for this for two reasons:
llama.cpp
in a single job.References:
CC @philiptaron @Green-Sky
Potential solutions
onPush
builds (.#checks
) less pureonSchedule
insteadQuestions
The text was updated successfully, but these errors were encountered: