Skip to content

Remove unused Vulnerability::Finding.for_pipelines_with_sha/batch_count_by_project_and_severity

What does this MR do?

This MR simply removes 2 unused methods that are causing problems for database decomposition.

These were discovered in !62092 (closed) .

These methods are cross-joining between ci_* and non ci_* tables and as such it will not be possible when ci_* tables are moved to a separate database.

Vulnerability::Finding.for_pipelines_with_sha is unused since 44a31d8b.

Vulnerability::Finding.batch_count_by_project_and_severity is unused since 3e1bc4f3

Screenshots or Screencasts (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #336732 (closed)

Edited by Dylan Griffith

Merge request reports