Skip to content

Begin populating the `has_vulnerability_resolution` column

What does this MR do and why?

Context

We need to be able to filter vulnerabilities on whether they have the "Resolve with Duo" button enabled.

This button is enabled if a finding's CWE value is included in this hard-coded list of CWE values.

We added a new column in !165548 (merged) to support this feature

This Commit

This commit starts populating the new column in the vulnerability ingestion flow.

You can see more detailed discussions in the epic

Follow-up work

Future MRs will address:

  • adding a batched background migration to backfill existing rows 1
  • Asynchronously adding an index 2
  • Synchronously adding the index

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.


epic: &15036 (closed)
resolves: #488394 (closed)

  1. Draft backfill MR: !166110 (merged)

  2. The index addition is being pushed off due to the difficulty of testing an index in database lab until the backfill is complete. This thread has more discussion

Edited by Michael Becker

Merge request reports

Loading