Skip to content

Remove securityScansSucceeded from DevOps Adoption

Dan Jensen requested to merge 335190-remove-devops-adoption-scanning-metric into master

What does this MR do?

The securityScansSucceeded query was causing a timeout error. Also, it is effectively being replaced with queries for individual scan types. This removes the column from the DevOps adoption table in the UI, and sets the value to always be false in the SnapshotCalculator (some value is necessary because the database column is NOT NULL). This does not remove the database column, because this is an urgent change, so that column (and remaining code) will be removed in a follow-up issue.

Note: We already tried to fix the timeout problem. We also considered rescuing query timeouts, but decided it would be better to simply remove the feature.

Screenshots

image image

Does this MR meet the acceptance criteria?

Conformity

Related to #335190 (closed)

Edited by Pavel Shutsin

Merge request reports

Loading