Skip to content
Snippets Groups Projects

Add policy name to ScanExecutionPolicyCommit mutation

Merged Sashi Kumar Kumaresan requested to merge sk/342695-update-policy-name into master
All threads resolved!

What does this MR do and why?

Addresses #342695 (closed)

This MR adds policy name to ScanExecutionPolicyCommit mutation which was introduced in !71655 (merged)

Screenshots or screen recordings

Kapture_2021-11-03_at_21.32.02

How to set up and validate locally

  1. Go to Security & Compliance -> Policies
  2. Create a new policy by clicking New Policy
  3. Give a name for the policy and click Create Merge Request and merge the created MR
  4. Go to Policies and edit the newly created policy and update the name of the policy

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 Sashi Kumar Kumaresan

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sashi Kumar Kumaresan deleted the sk/339794-update-policy-name-be branch. This merge request now targets the master branch

    deleted the sk/339794-update-policy-name-be branch. This merge request now targets the master branch

  • added 1545 commits

    Compare with previous version

  • changed milestone to %14.6

  • Bundle size analysis [beta]

    This compares changes in bundle size for entry points between the commits b2c40a05 and 35a53cae

    :sparkles: Special assets

    Entrypoint / Name Size before Size after Diff Diff in percent
    average 3.24 MB 3.24 MB - -0.0 %
    mainChunk 1.89 MB 1.89 MB - 0.0 %

    :tada: Significant Reduction: 1

    Expand
    Entrypoint / Name Size before Size after Diff Diff in percent
    pages.projects.environments.index 1.28 MB 1.25 MB -29.49 KB -2.3 %

    Note: We do not have exact data for b2c40a05. So we have used data from: 1f52a43b.
    The intended commit has no webpack pipeline, so we chose the last commit with one before it.

    Please look at the full report for more details


    Read more about how this report works.

    Generated by :no_entry_sign: Danger

  • added 1620 commits

    Compare with previous version

  • Sashi Kumar Kumaresan marked this merge request as ready

    marked this merge request as ready

  • Sashi Kumar Kumaresan marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

    marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

  • Sashi Kumar Kumaresan resolved all threads

    resolved all threads

  • requested review from @aturinske

  • Alexander Turinske approved this merge request

    approved this merge request

  • Alexander Turinske requested review from @ekigbo and removed review request for @aturinske

    requested review from @ekigbo and removed review request for @aturinske

  • :wave: @aturinske, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Ezekiel Kigbo approved this merge request

    approved this merge request

  • Ezekiel Kigbo changed milestone to %14.5

    changed milestone to %14.5

  • Ezekiel Kigbo resolved all threads

    resolved all threads

  • Ezekiel Kigbo enabled an automatic merge when the pipeline for e55cc9f2 succeeds

    enabled an automatic merge when the pipeline for e55cc9f2 succeeds

  • merged

  • Ezekiel Kigbo mentioned in commit fb95828f

    mentioned in commit fb95828f

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading