Skip to content
Snippets Groups Projects

Remove search_index_all_projects FF

Merged Terri Chu requested to merge 432489-remove-ff into master
All threads resolved!

What does this MR do and why?

Related to #432489 (closed)

Removes search_index_all_projects FF and releases the index all projects feature for Advanced search

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

N/A - not a frontend change

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Edited by Terri Chu

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
  • Ashraf Khamis removed review request for @ashrafkhamis

    removed review request for @ashrafkhamis

  • Terri Chu added 1 commit

    added 1 commit

    • 85a4162d - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • requested review from @ashrafkhamis

  • Ashraf Khamis approved this merge request

    approved this merge request

  • Ashraf Khamis removed review request for @ashrafkhamis

    removed review request for @ashrafkhamis

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 8e6ddc2b

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 51     | 0      | 2       | 0     | 53    | ✅     |
    | Create      | 77     | 0      | 9       | 0     | 86    | ✅     |
    | Monitor     | 7      | 0      | 0       | 0     | 7     | ✅     |
    | Govern      | 66     | 0      | 0       | 0     | 66    | ✅     |
    | Data Stores | 31     | 0      | 0       | 0     | 31    | ✅     |
    | Verify      | 35     | 0      | 1       | 0     | 36    | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Package     | 24     | 0      | 6       | 0     | 30    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 298    | 0      | 19      | 0     | 317   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :white_check_mark: test report for 8e6ddc2b

    expand test summary
    +---------------------------------------------------------------------+
    |                           suites summary                            |
    +----------------+--------+--------+---------+-------+-------+--------+
    |                | passed | failed | skipped | flaky | total | result |
    +----------------+--------+--------+---------+-------+-------+--------+
    | Package        | 205    | 0      | 38      | 0     | 243   | ✅     |
    | Manage         | 38     | 0      | 12      | 1     | 50    | ✅     |
    | Plan           | 249    | 0      | 19      | 0     | 268   | ✅     |
    | Verify         | 150    | 0      | 30      | 3     | 180   | ✅     |
    | Systems        | 10     | 0      | 0       | 0     | 10    | ✅     |
    | GitLab Metrics | 2      | 0      | 1       | 0     | 3     | ✅     |
    | Data Stores    | 119    | 0      | 28      | 0     | 147   | ✅     |
    | Configure      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Create         | 570    | 0      | 81      | 10    | 651   | ✅     |
    | Govern         | 271    | 0      | 20      | 0     | 291   | ✅     |
    | Monitor        | 36     | 0      | 13      | 0     | 49    | ✅     |
    | Analytics      | 7      | 0      | 0       | 0     | 7     | ✅     |
    | Fulfillment    | 8      | 0      | 75      | 1     | 83    | ✅     |
    | Release        | 15     | 0      | 3       | 0     | 18    | ✅     |
    | Secure         | 6      | 0      | 3       | 0     | 9     | ✅     |
    | Growth         | 0      | 0      | 6       | 0     | 6     | ➖     |
    | Ai-powered     | 0      | 0      | 3       | 0     | 3     | ➖     |
    | ModelOps       | 0      | 0      | 3       | 0     | 3     | ➖     |
    +----------------+--------+--------+---------+-------+-------+--------+
    | Total          | 1687   | 0      | 344     | 15    | 2031  | ✅     |
    +----------------+--------+--------+---------+-------+-------+--------+
  • :tools: Generated by gitlab_quality-test_tooling.


    :snail: Slow tests detected in this merge request. These slow tests might be related to this merge request's changes.

    Click to expand
    Job File Name Duration Expected duration
    #6511659267 spec/features/commit_spec.rb#L68 Commit behaves like single commit view pagination shows paginated content and controls to navigate 64.52 s < 50.13 s
    #6511659167 spec/features/commit_spec.rb#L68 Commit behaves like single commit view pagination shows paginated content and controls to navigate 64.44 s < 50.13 s
  • A deleted user added rspec:slow test detected label
  • Terri Chu marked this merge request as ready

    marked this merge request as ready

  • Terri Chu added 348 commits

    added 348 commits

    Compare with previous version

  • Terri Chu requested review from @rkumar555

    requested review from @rkumar555

  • Ravi Kumar approved this merge request

    approved this merge request

  • Ravi Kumar requested review from @maddievn and removed review request for @rkumar555

    requested review from @maddievn and removed review request for @rkumar555

  • Madelein van Niekerk
  • Madelein van Niekerk
  • Thanks for removing the flag. I had two minor questions about the delete_project flag :ping_pong:

  • Madelein van Niekerk approved this merge request

    approved this merge request

  • Madelein van Niekerk resolved all threads

    resolved all threads

  • Madelein van Niekerk enabled an automatic merge when the pipeline for f8eee93b succeeds

    enabled an automatic merge when the pipeline for f8eee93b succeeds

  • mentioned in commit be05eb41

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading