Skip to content

Add scan duration check to SAST template

Igor Frenkel requested to merge 196697-add-scan-duration-check-to-sast into master

What does this MR do?

This MR adds a scan duration check to the qa template for sast. It also adds a new variable SCAN_DURATION_MARGIN_PERCENT for flexibility in alerting on scan duration that exceeds MAX_SCAN_DURATION_SECONDS. This job allows more precise testing of scan duration by reading the start and end times of the report. And it allows testing to become more granular by allowing a test for each downstream project. I added an example run to gosec: https://gitlab.com/gitlab-org/security-products/analyzers/gosec/-/pipelines/215535529

The 3 new downstream jobs are only there for review in this MR (the branch will not be merged) showing:

The analyzer branch points to the downstream branch for go-modules which in turn points to this branch in ci-templates.

Once this MR is merged, the process will be to set the variable to 0 (in a test branch) and trigger a pipeline in each SAST analyzer so as to get a more precise reading for each downstream project. After this an MR can be created with the individual test values.

What are the relevant issue numbers?

gitlab-org/gitlab#196697 (closed)

Does this MR meet the acceptance criteria?

Edited by Igor Frenkel

Merge request reports