Skip to content

ci(coverity): fix Coverity scans #5266

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

Merged
merged 1 commit into from
Aug 14, 2025
Merged

Conversation

alex-ter
Copy link
Contributor

@alex-ter alex-ter commented Aug 9, 2025

Starting in version 2024.12, the "cov-build" command is only supposed to be used for "build capture", while the "filesystem capture" used for non-compiled languages is only available via Coverity CLI (the "coverity" command).

The GH Action we use does not yet support that, so introduce a workaround by tricking the Action to run the correct command via command injection.

Tested in my fork: example run.

I've also explicitly limited the languages to Python and JS, as it was needlessly picking many documentation and other aux files. Let me know if you think this should be tweaked. That also removed the need for one of the exclude regexes.

Fixes #5254.

Starting in version 2024.12, the "cov-build" command is only
supposed to be used for "build capture", while the "filesystem capture"
used for non-compiled languages is only available via Coverity CLI
(the "coverity" command).

The GH Action we use does not yet support that, so introduce a
workaround by tricking the Action to run the correct command via command
injection.

Fixes intel#5254.

Signed-off-by: Alex T. <[email protected]>
Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hah, I love it. Let's give this a shot, but also I'm not particularly set on using the workflow we're using if this doesn't work and it turns out to be easier to just call coverity directly.

Coverity doesn't run on PRs so I'll merge this and we'll see what happens.

@terriko terriko merged commit 83ec689 into intel:main Aug 14, 2025
38 of 39 checks passed
@alex-ter alex-ter deleted the gh5254-fix-cov-scan branch August 15, 2025 14:24
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

Successfully merging this pull request may close these issues.

ci: coverity scan is failing
2 participants