actions: Address some of the TICS failures #988
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
claudiubelu
force-pushed
the
tics
branch
3 times, most recently
from
1a9267e to
73f8101
Compare
There are some errors in the TICS job, even though the job passes. Some of these errors relate to some missing dependencies for pylint. There is one other notable type of error (hundreds of errors), related to Coverity: [ERROR 5336] No analysis data found for file '/home/runner/work/k8s-snap/k8s-snap/src/k8s/pkg/k8sd/controllers/csrsigning/const.go' in 'Coverity'. Please check if the file is analyzed properly, and whether the file is in the analysis scope of the tool. These errors are currently unsolvable. The current version of TICS comes with the Coverity 2024.9.0 version (as seen in cov-build-console.log): Coverity Build Capture (64-bit) version 2024.9.0 on Linux 6.8.0-1020-azure x86_64 Internal version numbers: 950e903b4e p-2024.9-push-30 According to the Coverity documentation [1], this version of Coverity only supports golang 1.21 and 1.22, while this project is built on golang 1.23. Coverity 2024.12.0 supports golang 1.23 as well. [1] https://documentation.blackduck.com/bundle/coverity-on-polaris-UAT/page/topics/r_coverity-compatible-platforms_2024.9.html
berkayoz
reviewed
Jan 31, 2025
| pip install requests jinja2 semver | ||
|
|
||
| # Integration tests are importing a local test_util module. Needed for pylint. | ||
| export PYTHONPATH="$(pwd)/tests/integration/tests" |
There was a problem hiding this comment.
Should we append this to the pythonpath instead of overriding it completely?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There are some errors in the TICS job, even though the job passes. Some of these errors relate to some missing dependencies for
pylint.There is one other notable type of error (hundreds of errors), related to Coverity:
These errors are currently unsolvable. The current version of TICS comes with the Coverity 2024.9.0 version (as seen in
cov-build-console.log):According to the Coverity documentation [1], this version of Coverity only supports golang 1.21 and 1.22, while this project is built on golang 1.23. Coverity 2024.12.0 supports golang 1.23 as well.
[1] https://documentation.blackduck.com/bundle/coverity-on-polaris-UAT/page/topics/r_coverity-compatible-platforms_2024.9.html