Skip to content

Add tags input box for scan execution actions

Alexander Turinske requested to merge 382994-include-tags into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Add tags input box for scan execution actions

  • introduce scan_execution_tags feature flag
  • add tags input box for users to input tags
  • add tests

Changelog: added

EE: true

Screenshots or screen recordings

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

Scenario w/out ff w/ ff
DAST - close up close_up image
DAST - with preview dast_without Screen_Shot_2022-11-27_at_13.43.09
Other scanner - preview sast_without Screen_Shot_2022-11-27_at_13.43.23

How to set up and validate locally

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

  1. Upload a GitLab Ultimate license
  2. Enable the echo "Feature.enable(:scan_execution_tags)" | rails c
  3. Navigate to http://gdk.test:3443/groups/flightjs/-/security/policies/new?type=scan_execution_policy
  4. Verify tags
  5. Navigate to http://gdk.test:3443/flightjs/Flight/-/security/policies/new?type=scan_execution_policy
  6. Verify tags

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

Edited by Alexander Turinske

Merge request reports