[CI] [GHA] Remove redundant args from the coverity command #2761
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes redundant arguments from the Coverity static analysis command in the GitHub Actions workflow to simplify the build process.
- Removed the
--fs-capture-search ./openvino.genaiargument from thecov-buildcommand
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| - name: Build for coverity | ||
| run: | | ||
| ${{ env.COV_TOOL_DIR }}/bin/cov-build --config ${{ env.COV_TOOL_DIR }}/config/coverity_config.xml --tmpdir cov_temp --dir ${BUILD_DIR}/cov-int --fs-capture-search ./openvino.genai sh build.sh | ||
| ${{ env.COV_TOOL_DIR }}/bin/cov-build --config ${{ env.COV_TOOL_DIR }}/config/coverity_config.xml --tmpdir cov_temp --dir ${BUILD_DIR}/cov-int sh build.sh |
There was a problem hiding this comment.
It's needed to include python configured by cov-configure --python --no-capture-config-files --version 3
There was a problem hiding this comment.
What is needed? --fs-capture-search or ./openvino.genai? If the former, then there should be another argument because
Since I do not have the admin rights to any Coverity project, I cannot check the command-line docs. If you could, please provide a replacement argument.
There was a problem hiding this comment.
I've invited you to the project. Let me know if you need secrets.COVERITY_SECRET_TOKEN value.
They deprecated --fs-capture-search https://documentation.blackduck.com/bundle/coverity-docs-2024.6/page/coverity-analysis/topics/moving_from_filesystem_capture_to_the_coverity_cli.html Will you be able to update the workflow to use the new tool?
There was a problem hiding this comment.
Yes, that is why I removed the argument.
As I understand, the latest available version is installed every time. Because of that, the Coverity pipelines started to fail when a newer version was installed with the argument deprecated.
There was a problem hiding this comment.
Unfortunately, the sole invite to a project is not enough:
Admin access is required.
Should fix https://github.com/openvinotoolkit/openvino.genai/actions/runs/17932277706/job/50991740711