Skip to content

DRAFT: First pass at adding license scan result policy rules behind feature flag

- requested to merge 371506-license-scanning-initial-ui into master

What does this MR do and why?

Adds the first iteration of license scanning result policies behind a feature flag.

Covers the following 2 issues:

#371506 (closed)

#371507 (closed)

Things done in this MR:

  • Refactor several Vue components to extract shared logic between license scan policies and security scan polices
  • Add 2 versions of configuration behind a feature flag.
  • Create a V2 feature flag component so we can more easily toggle between the two.

NOTE: %{licenseType} will be completed as part of #371508 (closed) next milestone

Slightly outdated youtube walkthrough, but gives you a general overview of the functionality: https://www.youtube.com/watch?v=5NJncwtLxos

Before After Info
Screen_Shot_2022-11-15_at_12.48.40_AM Screen_Shot_2022-11-15_at_12.43.56_AM Initial State
Screen_Shot_2022-11-15_at_12.48.17_AM Screen_Shot_2022-11-15_at_12.04.03_AM After adding additional rules

Verification Steps

  • In a rails console enable Feature.enable(:license_scanning_policies)
  • As a EE user Navigate to http://gdk.test:3000/<group>/<project>/-/security/policies
  • Click New Policy button
  • On the Scan Result Policy card click Select policy
  • Add a rule

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 #371506 (closed)

Edited by -

Merge request reports