-
Notifications
You must be signed in to change notification settings - Fork 141
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
sast-coverity-check: added stats for Coverity scans #1693
Conversation
This MR was discussed here: jperezdealgaba#4 |
edece1a
to
b28adb5
Compare
@tnevrlka Would you mind enabling this pipeline please? |
/ok-to-test |
b28adb5
to
5903cf2
Compare
task/sast-coverity-check-oci-ta/0.1/sast-coverity-check-oci-ta.yaml
Outdated
Show resolved
Hide resolved
5903cf2
to
2c9cb0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Solves: https://issues.redhat.com/browse/OSH-769 Adding the stats to Coverity scans in the result's SARIF file of successful scans.
/ok-to-test |
@kasemAlem @dirgim Could you please check what happened that
I do not have access to the corresponding logs myself. |
@kdudka Seems like the PLR logs are removed , can I rebase and rerun then we follow result ? |
@kasemAlem Yes, please. Note that I rebased this pull request 4 hours ago. It did not help. |
LGTM |
/ok-to-test |
Solves: https://issues.redhat.com/browse/OSH-769
Adding the stats to Coverity scans in the result's SARIF file of successful scans.
Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.