Skip to content
Snippets Groups Projects

Use new vulnerability report for pipeline security tab

Merged Daniel Tian requested to merge 347155-use-new-vuln-report-for-pipeline into master

What does this MR do and why?

This MR switches the GraphQL version of the pipeline security tab vulnerability report, which is currently using an old version of a shared vulnerability report, to use the new version. The new version is already used by all the other report levels (project, group, and instance), and this is the last one that needs to be switched over. This MR also removes a large number of unused components (and their tests) now that the old shared vulnerability report is completely deprecated.

Note that the GraphQL version of the report on the pipeline security tab is still under development, so you will need to enable the feature flag :pipeline_security_dashboard_graphql to see it. Otherwise, you will see the Vuex version (the default).

Vuex version (if you see this, you need to enable the feature flag) GraphQL old version GraphQL new version, note that the counts aren't shown because the GraphQL endpoint doesn't exist for it yet
ksnip_20220307-230455 ksnip_20220307-225951 ksnip_20220307-225822

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #347155 (closed)

Edited by Daniel Tian

Merge request reports

Merged results pipeline #492514142 passed with warnings

Pipeline: GitLab

#492535110

    Pipeline: omnibus-gitlab-mirror

    #492535874

      Merged results pipeline passed with warnings for f9573640

      Test coverage 86.81% (-1.47%) from 2 jobs

      Merged by Savas VedovaSavas Vedova 3 years ago (Mar 15, 2022 1:21pm UTC)

      Loading

      Activity

      Filter activity
      • Approvals
      • Assignees & reviewers
      • Comments (from bots)
      • Comments (from users)
      • Commits & branches
      • Edits
      • Labels
      • Lock status
      • Mentions
      • Merge request status
      • Tracking
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      Please register or sign in to reply
      Loading