Skip to content

Fixes by removing cluster_image_scanning as part

Zamir Martins requested to merge fix_cluster_image_scanning_added_as_part_of_constant into master

What does this MR do and why?

Fixes by removing cluster_image_scanning as part of the array of keys. This bug was introduced while refactoring throughout MR review.

Related documentation: https://docs.gitlab.com/ee/user/application_security/#vulnerability-check-rule

Screenshots or screen recordings

Selecting all scanners is correct

Screen_Shot_2021-11-02_at_6.21.03_PM

However the array of keys still includes it so it fails backend validation

Screen_Shot_2021-11-02_at_6.21.30_PM

How to set up and validate locally

  1. Open vulnerability-check dialog as per documentation
  2. Click the option Select all for the security scanners dropdown.
  3. Click on Create/Update button. !. There will be a validation error as per screenshot (if not on this branch).

MR acceptance checklist

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

Edited by Zamir Martins

Merge request reports