Skip to content
Snippets Groups Projects

Enable invalid_scan_result_policy_prevents_merge by default

What does this MR do and why?

This MR enables invalid_scan_result_policy_prevents_merge feature flag by default. The feature flag has been rolled out globally throughout 16.0 and there has been no concerning feedback.

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

Edited by Martin Čavoj

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
  • Contributor
    1 Message
    :book: This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge.

    Documentation review

    The following files require a review from a technical writer:

    The review does not need to block merging this merge request. See the:

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • Contributor

    Allure report

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :exclamation: test report for 7dd40d5e

    expand test summary
    +-----------------------------------------------------------------------+
    |                            suites summary                             |
    +------------------+--------+--------+---------+-------+-------+--------+
    |                  | passed | failed | skipped | flaky | total | result |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Govern           | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Create           | 8      | 0      | 1       | 0     | 9     | ✅     |
    | Data Stores      | 2      | 0      | 0       | 1     | 2     | ❗     |
    | Plan             | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Manage           | 1      | 0      | 0       | 0     | 1     | ✅     |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Total            | 21     | 0      | 2       | 1     | 23    | ❗     |
    +------------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :x: test report for 7dd40d5e

    expand test summary
    +-------------------------------------------------------------+
    |                       suites summary                        |
    +--------+--------+--------+---------+-------+-------+--------+
    |        | passed | failed | skipped | flaky | total | result |
    +--------+--------+--------+---------+-------+-------+--------+
    | Govern | 96     | 2      | 2       | 10    | 100   | ❌     |
    +--------+--------+--------+---------+-------+-------+--------+
    | Total  | 96     | 2      | 2       | 10    | 100   | ❌     |
    +--------+--------+--------+---------+-------+-------+--------+
  • Sashi Kumar Kumaresan requested review from @bala.kumar and removed review request for @sashi_kumar

    requested review from @bala.kumar and removed review request for @sashi_kumar

  • Sashi Kumar Kumaresan approved this merge request

    approved this merge request

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

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Martin Čavoj added 1 commit

    added 1 commit

    • af6dff4c - Enable invalid_scan_result_policy_prevents_merge by default

    Compare with previous version

  • Martin Čavoj added 1 commit

    added 1 commit

    • 155b17f4 - Enable invalid_scan_result_policy_prevents_merge by default

    Compare with previous version

  • A deleted user added documentation label

    added documentation label

  • Please register or sign in to reply
    Loading