-
-
Notifications
You must be signed in to change notification settings - Fork 327
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
CI: configure Coverity Scan with debug flag #3572
Conversation
Configure and build for Coverity Scan with debug flag set. Also run `cov-build` directly with `make` as argument. In addition some style fixes to pass pre-commit.
I'll let you see if the scans work as expected |
Now we’ll see… |
Appears to work fine:
but still no update on Coverity Scan. |
I logged to the coverity account I created when I got invited to the coverity project. I do not see it either. In fact last scan in 2022. The only thing I see is at the end of uploading the results, it mentions to make sure cov-build version matches 2023.6.2 or 2022.12.2, but at the top of the build, we have 2023.6.2 written, so they should match. |
@wenzeslaus In case you missed this, do you have any info on why the Coverity fails? |
Unfortunately, I don't. I just see what others (with access) see in the Coverity scan tool online. There is no log of failed uploads or anything like that. I suggested to @jadenabrams100 yesterday to create an artifact in the workflow which we can manually upload as something which can be done in the short time the team has left. @jadenabrams100 will also try to contact Coverity about that. |
I have this working, with nilason@f01b89a on my fork. |
@nilason I changed your role on scan.coverity.com to Maintainer/Owner (aka Admin). In the past, the an email was triggered by a successful upload and analysis, there is no successful upload here as far as the project on the website is concerned, so no email makes sense. |
I have received an update back from an admin at Synopsys. The admin said that the fault is likely due to scanning results being over the 500MB limit, so the standard submission process won't cut it. Fortunately, @wenzeslaus and I discovered instructions for how to run automated scans for projects over 500MB. I will get to work on implementing the new method as soon as I can in addition to saving the scans as an artifact. |
Thanks!
Yes, I get an email for each successful upload (so far all of them were, so I don't know what will happen by failure). |
How do they count the size? The uploaded package is about 45 MB. |
According to the admin, they count the size as the size of the resulting I have completed the changes and submitted them in #3606. |
Configure and build for Coverity Scan with debug flag set. Also run
cov-build
directly withmake
as argument.In addition some style fixes to pass pre-commit.
Attempt to fix (still broken?) Coverity Scan.