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
All threads resolved!

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
  • Contributor
    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

  • Contributor

    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   | ❗     |
    +--------+--------+--------+---------+-------+-------+--------+
  • 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

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

  • mo khan mentioned in merge request !127045 (merged)

    mentioned in merge request !127045 (merged)

  • Brian Williams resolved all threads

    resolved all threads

  • Brian Williams enabled an automatic merge when the pipeline for 7bf47ca0 succeeds

    enabled an automatic merge when the pipeline for 7bf47ca0 succeeds

  • merged

  • Brian Williams mentioned in commit d13a1b5b

    mentioned in commit d13a1b5b

  • mo khan mentioned in issue #417486 (closed)

    mentioned in issue #417486 (closed)

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading