Skip to content
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

Coverity no longer accepts reports from CentOS7 containers because the coverity version is too old #6

Closed
tmadlener opened this issue Dec 15, 2023 · 8 comments

Comments

@tmadlener
Copy link
Contributor

This is potentially related to #5. Should simply switching a coverity workflow to use an el9 based LCG release work?

@andresailer
Copy link
Member

I already changed that 16f38a9 ? (I don't even remember...)

@andresailer
Copy link
Member

Yes, switching the workflows to use the el9 instead of latest tag for the container should be it.

@tmadlener
Copy link
Contributor Author

Not quite yet it seems: https://github.com/AIDASoft/podio/actions/runs/7246402281/job/19738267861#step:4:124

I am not entirely sure why though, because readline-devel should be in the installed packages for the e9 image:

readline-devel

@andresailer
Copy link
Member

It still used the latest container instead of el9

@tmadlener
Copy link
Contributor Author

So, do we push the el9 based coverity container to latest, or should I explicitly ask for coverity:el9 as container in the workflow (that I have to touch in any case to update the LCG release to an el9 based one)?

@andresailer
Copy link
Member

Now that the centos7 coverity build silently fails, we might as well push el9 to latest.

@andresailer
Copy link
Member

latest now points to el9

@tmadlener
Copy link
Contributor Author

And at least podio has working coverity scans again :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants