Remove use_marker_ranges feature_flag
What does this MR do and why?
Removes use_marker_ranges feature_flag.
This flag has been enabled for a while: #324638 (comment 550455214)
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.
Related to #391508 (closed)
Merge request reports
Activity
changed milestone to %15.11
assigned to @j.seto
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 Pedro Pombeiro (
@pedropombeiro
) (UTC+2, 6 hours ahead of@j.seto
)Niko Belokolodov (
@nbelokolodov
) (UTC+13, 17 hours ahead of@j.seto
)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
Dangermentioned in commit gitlab-org-sandbox/gitlab-jh-validation@dd6b0d0e
Allure report
allure-report-publisher
generated test report!e2e-package-and-test:
test report for 555eb2d9expand test summary
+-----------------------------------------------------------------------+ | suites summary | +------------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +------------------+--------+--------+---------+-------+-------+--------+ | Plan | 311 | 0 | 0 | 117 | 311 | ❗ | | Monitor | 48 | 0 | 0 | 48 | 48 | ❗ | | Manage | 162 | 0 | 15 | 90 | 177 | ❗ | | Verify | 259 | 0 | 45 | 254 | 304 | ❗ | | Fulfillment | 10 | 0 | 110 | 0 | 120 | ✅ | | Release | 30 | 0 | 0 | 25 | 30 | ❗ | | Govern | 234 | 0 | 0 | 234 | 234 | ❗ | | Package | 125 | 0 | 23 | 125 | 148 | ❗ | | Configure | 0 | 0 | 15 | 0 | 15 | ➖ | | Data Stores | 174 | 0 | 0 | 60 | 174 | ❗ | | Growth | 0 | 0 | 10 | 0 | 10 | ➖ | | Analytics | 10 | 0 | 0 | 10 | 10 | ❗ | | Secure | 35 | 0 | 25 | 35 | 60 | ❗ | | Create | 824 | 5 | 37 | 350 | 866 | ❌ | | Framework sanity | 0 | 0 | 7 | 0 | 7 | ➖ | | ModelOps | 0 | 0 | 5 | 0 | 5 | ➖ | | Systems | 2 | 0 | 0 | 0 | 2 | ✅ | +------------------+--------+--------+---------+-------+-------+--------+ | Total | 2224 | 5 | 292 | 1348 | 2521 | ❌ | +------------------+--------+--------+---------+-------+-------+--------+
changed milestone to %15.10
- Resolved by Vasilii Iakliushin
requested review from @avielle
- Resolved by Avielle Wolfe
removed review request for @avielle
@avielle
, 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:
added pipeline:mr-approved label
changed milestone to %15.11
added 1261 commits
-
145ab2fe...0ac2f3e9 - 1260 commits from branch
master
- 3c7b5ec9 - Remove use_marker_ranges feature_flag
-
145ab2fe...0ac2f3e9 - 1260 commits from branch
mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@b2dea7ea
- Resolved by Vasilii Iakliushin
hi @vyaklushin
would you mind reviewing this for backend as maintainer?
requested review from @vyaklushin
- Resolved by Jerry Seto
removed review request for @vyaklushin
mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@641cab8f
requested review from @vyaklushin
mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@135c3c82
marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed
enabled an automatic merge when the pipeline for e2eb947d succeeds
mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@e2eb947d
mentioned in commit 84c1de66
added workflowstaging-canary label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-staging label and removed workflowproduction label
added workflowpost-deploy-db-production label and removed workflowpost-deploy-db-staging label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label