Skip to content
Snippets Groups Projects

Enable descendant_security_scans by default

Merged mo khan requested to merge mokhax/417486/enable-ff-by-default into master

What does this MR do and why?

This change enables descendant_security_scans feature flag by default.

#417486 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

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
  • 1 Message
    :book: CHANGELOG missing:

    If you want to create a changelog entry for GitLab FOSS, add the Changelog trailer to the commit message you want to add to the changelog.

    If you want to create a changelog entry for GitLab EE, also add the EE: true trailer to your commit message.

    If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

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

    Generated by :no_entry_sign: Danger

    Edited by Ghost User
  • Allure report

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :exclamation: test report for 2ee0c132

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Govern      | 21     | 0      | 0       | 18    | 21    | ❗     |
    | Plan        | 47     | 0      | 0       | 40    | 47    | ❗     |
    | Create      | 19     | 0      | 0       | 18    | 19    | ❗     |
    | Data Stores | 20     | 0      | 0       | 15    | 20    | ❗     |
    | Verify      | 8      | 0      | 0       | 8     | 8     | ❗     |
    | Manage      | 12     | 0      | 1       | 12    | 13    | ❗     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 127    | 0      | 1       | 111   | 128   | ❗     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :exclamation: test report for 2ee0c132

    expand test summary
    +-------------------------------------------------------------+
    |                       suites summary                        |
    +--------+--------+--------+---------+-------+-------+--------+
    |        | passed | failed | skipped | flaky | total | result |
    +--------+--------+--------+---------+-------+-------+--------+
    | Govern | 102    | 0      | 8       | 8     | 110   | ❗     |
    +--------+--------+--------+---------+-------+-------+--------+
    | Total  | 102    | 0      | 8       | 8     | 110   | ❗     |
    +--------+--------+--------+---------+-------+-------+--------+
    Edited by Ghost User
  • mo khan mentioned in issue #393305 (closed)

    mentioned in issue #393305 (closed)

  • mo khan marked this merge request as ready

    marked this merge request as ready

  • added typemaintenance label and removed bugfunctional typebug labels

  • mo khan added 484 commits

    added 484 commits

    Compare with previous version

  • mo khan requested review from @bwill

    requested review from @bwill

  • Brian Williams approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading