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)
-
Draft backfill MR: !166110 (merged)
↩ -
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
↩