-
Notifications
You must be signed in to change notification settings - Fork 60
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
Use run-lcg-view github action and switch to more recent LCG releases in CI #182
Conversation
If you are contemplating the CI, it would be also good to redefine the CI jobs with the run-lcg-view Action |
please be patient with the CI failures, we have a ticket open with IIS Web Hosting Service there are some issues with the EOS mount on the machines that host the webpage where the cvmfs binaries are fetched from Edit: s/month/mount/ |
Alright. Thanks for the heads up. |
No, we are very far away from saturating the server, it's not that kind of a problem |
I just realized that the required checks have to be adapted after merging this, since they have the LCG release in their name. Currently this is waiting for jobs to finish, which no longer exist after this PR. |
I think you can still merge |
but if you keep them separate you can rerun them separately |
It is really annoying that they have still not implemented single job restarts |
So, I think it is best to keep it like this until single job restarts are a possibility, to at least have some sort of granularity. |
From the meeting discussion: Use only LCG_99, maybe also dev3 (root master) and dev4 (usually latest release patch branch) |
Keep LCG_98 to have clang11 still in the mix
The CVMFS issues should be fixed, the cause was:
And as far as I understand the EOS head nodes should have received the patch today at 10. |
I was to quick, they fixed another problem with EOS that also causes 403 but not this one
|
@petricm are there any news on the cvmfs issue? If it is fixed, I think this could be merged and the settings adjusted accordingly, so that also the other PRs can run against an updated CI again. |
It is merged, but they did not say that it was deployed. But you can merge this regardless, as podio anyway used the cvmfs action already, so with or without this PR the CI is equally affected. |
For the cvmfs errors, I see that as well in the ci of the key4hep packages, and there is about a 1/50 chance. So good to have it fixed, but this is fine to merge I think, as it is easily restarted. Thanks to Marko, this action can now also use the /cvmfs/sw.hsf.org installation, I'll set this up in a future pr. |
The fix for EOS was deployed |
After our discussion during yesterdays meeting I have now also added an additional workflow that builds on top of the latest key4hep release. I think with this we really have everything that we would like to have in this PR. |
BEGINRELEASENOTES
run-lcg-view
github action and switch to more recent LCG releases to run CI.ENDRELEASENOTES
After this the builds are:
gcc8
,clang10
), dev3 (clang10
), dev4 (gcc10
,clang10
)ENABLE_SIO=ON
gcc10
)ENABLE_SIO=ON
gcc9
)ENABLE_SIO=ON
clang120
), 98python3 (clang110
)gcc10
)The most important change is that I have removed all builds that use python2, so that all of our CI is now using python3 exclusively. I have also removed the dedicated workflow file for the sio tests and instead integrated that into the other workflows.
Now that we have coverity working again, maybe we can also advertise our CI a bit more prominently ;)
Should we add all badges, i.e. the sio build and the python checks as well?