Cleanup use_gitaly_pagination_for_refs feature flag
What does this MR do and why?
Removes all references to the use_gitaly_pagination_for_refs feature flag.
Issue: #375737 (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.
-
I have evaluated the MR acceptance checklist for this MR.
Merge request reports
Activity
changed milestone to %15.5
assigned to @ghinfey
Suggested Reviewers (beta)
The individuals below may be good candidates to participate in the review based on various factors.
You can use slash commands in comments to quickly assign
/assign_reviewer @user1
.Suggested Reviewers @rspeicher
,@rymai
,@psimyn
,@marcel.amirault
,@tkuah
If you do not believe these suggestions are useful, please apply the label Bad Suggested Reviewer. You can also provide feedback for this feature on this issue:
https://gitlab.com/gitlab-org/gitlab/-/issues/357923
.Automatically generated by Suggested Reviewers Bot - an experimental ML-based recommendation engine created by ~"group::applied ml".
- A deleted user
added backend database databasereview pending labels
Reviewer roulette
Changes that require review have been detected!
Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:
Category Reviewer Maintainer backend Bishwa Hang Rai ( @bhrai
) (UTC+2)James Fargher ( @proglottis
) (UTC+13)To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.
To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.
Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Ghost User- Resolved by 🤖 GitLab Bot 🤖
Proper labels assigned to this merge request. Please ignore me.
@ghinfey - please see the following guidance and update this merge request.1 Error, 1 Warning Please add typebug typefeature, or typemaintenance label to this merge request. Please add a subtype label to this merge request. If you have added a type label and do not feel the purpose of this merge request matches one of the subtypes labels, please resolve this discussion.
Edited by 🤖 GitLab Bot 🤖
Allure report
allure-report-publisher
generated test report!e2e-review-qa:
test report for d322db35expand test summary
+-----------------------------------------------------------------------------------------+ | suites summary | +------------------------------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +------------------------------------+--------+--------+---------+-------+-------+--------+ | Manage | 52 | 0 | 8 | 13 | 60 | ❗ | | Create | 154 | 0 | 11 | 25 | 165 | ❗ | | Plan | 47 | 0 | 1 | 0 | 48 | ✅ | | Verify | 12 | 0 | 1 | 4 | 13 | ❗ | | Configure | 0 | 0 | 1 | 0 | 1 | ➖ | | Protect | 2 | 0 | 0 | 1 | 2 | ❗ | | Feature flag handler sanity checks | 9 | 0 | 0 | 0 | 9 | ✅ | | Secure | 2 | 0 | 0 | 2 | 2 | ❗ | | Version sanity check | 0 | 0 | 1 | 0 | 1 | ➖ | | Package | 0 | 0 | 1 | 0 | 1 | ➖ | +------------------------------------+--------+--------+---------+-------+-------+--------+ | Total | 278 | 0 | 24 | 45 | 302 | ❗ | +------------------------------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for d322db35expand test summary
+---------------------------------------------------------------------------+ | suites summary | +----------------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +----------------------+--------+--------+---------+-------+-------+--------+ | Manage | 58 | 32 | 13 | 38 | 103 | ❌ | | Create | 123 | 35 | 7 | 44 | 165 | ❌ | | Plan | 33 | 24 | 0 | 31 | 57 | ❌ | | Verify | 35 | 8 | 8 | 8 | 51 | ❌ | | Fulfillment | 1 | 1 | 14 | 1 | 16 | ❌ | | Secure | 1 | 22 | 0 | 23 | 23 | ❌ | | Protect | 1 | 1 | 0 | 2 | 2 | ❌ | | Release | 4 | 0 | 0 | 0 | 4 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Configure | 0 | 0 | 3 | 0 | 3 | ➖ | | Version sanity check | 0 | 0 | 1 | 1 | 1 | ➖ | | Package | 0 | 0 | 3 | 0 | 3 | ➖ | +----------------------+--------+--------+---------+-------+-------+--------+ | Total | 258 | 123 | 49 | 148 | 430 | ❌ | +----------------------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Usermentioned in commit cf7d650f
added 1 commit
- cf7d650f - Remove use_gitaly_pagination_for_refs feature flag
added devopscreate groupsource code maintenancerefactor sectiondev typemaintenance workflowpost-deploy-db-staging labels and removed backend database databasereview pending labels
removed workflowpost-deploy-db-staging label
added backend feature workflowpost-deploy-db-production labels
removed feature label
added feature flag + 1 deleted label
mentioned in commit 35ec2e65
added 471 commits
-
cf7d650f...5675a837 - 470 commits from branch
master
- 35ec2e65 - Remove use_gitaly_pagination_for_refs feature flag
-
cf7d650f...5675a837 - 470 commits from branch
- Resolved by Gavin Hinfey