Skip to content
Snippets Groups Projects

Add request specs for security policies mutations

All threads resolved!

What does this MR do and why?

In this MR we're adding missing specs for recently added security policies mutations.

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

Edited by Alan (Maciej) Paruszewski

Merge request reports

Merged results pipeline #429733699 passed

Merged results pipeline passed for 69e8c88e

Test coverage 86.76% (-1.44%) from 2 jobs
Approved by

Merged by Thong KuahThong Kuah 3 years ago (Dec 15, 2021 3:13am UTC)

Merge details

  • Changes merged into master with 9ae8bef2 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #429799843 failed

Pipeline failed for 9ae8bef2 on master

Test coverage 78.57% (-1.44%) from 2 jobs
8 environments impacted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah
  • Thong Kuah removed review request for @tkuah

    removed review request for @tkuah

  • added 1 commit

    Compare with previous version

  • Ghost User
  • requested review from @tkuah

  • Thong Kuah resolved all threads

    resolved all threads

  • Thong Kuah approved this merge request

    approved this merge request

  • Thong Kuah unapproved this merge request

    unapproved this merge request

  • Thong Kuah approved this merge request

    approved this merge request

  • Thong Kuah unapproved this merge request

    unapproved this merge request

  • Thong Kuah approved this merge request

    approved this merge request

  • :wave: @tkuah, 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:

  • Thong Kuah enabled an automatic merge when the pipeline for 69e8c88e succeeds

    enabled an automatic merge when the pipeline for 69e8c88e succeeds

  • merged

  • Thong Kuah mentioned in commit 9ae8bef2

    mentioned in commit 9ae8bef2

  • added workflowstaging label and removed workflowin review label

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading