Closed Bug 1555621 Opened 5 years ago Closed 5 years ago

[Static-Analysis][Coverity] In order to have review-time static-analysis based on Coverity we need to have a task in mach

Categories

(NSS :: Test, task, P2)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: andi, Assigned: andi)

References

Details

Attachments

(2 files)

Since we want to implement our review-time static-analysis from firefox to nss we need to have a task in the library's mach that implements this sort of analysis.
This feature adds two options:

  • Run the analysis locally having a config file for Coverity, this config file can be obtained right after the developer has been granted access to out coverity connect platform where we also store the nss project
  • Run the analysis as a try job for the nss project, later on this try job will generate an artifact that the releng-services bot can use it in order to post defects to phabricator.
Component: Libraries → Test
Priority: -- → P2
QA Contact: jjones → mwobensmith

The configuration file must look like this:

server_url: ip
auth_key: some_key
package_url: 'ip/cov-package.tar.gz'
package_name: cov-package
package_ver: vXXYYZZ

On the try jobs this is automatically downloaded from the associated secret. If any of the reviewers wants to try this I can grant access to our Coverity Platform server from where he can download the config file

Attachment #9068647 - Attachment description: Bug 1555621 - add Coverity as static-analysis target to NSS. → Bug 1555621 - add Coverity as static-analysis target in `mach`.

Depends on D33138

Depends on: 1556972
Summary: [Static-Analysis][Coverity] In order t have review-time static-analysis based on Coverity we need to have a task in mach → [Static-Analysis][Coverity] In order to have review-time static-analysis based on Coverity we need to have a task in mach
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: