Remove integrated_error_tracking feature flag
What does this MR do?
Remove integrated_error_tracking
feature flag. It was used for development and is no longer needed now when a feature is working.
- feature issue => #329596 (closed)
- feature flag issue => #335846 (closed)
Does this MR meet the acceptance criteria?
Conformity
-
I have included changelog trailers, or none are needed. (Does this MR need a changelog?) -
I have added/updated documentation, or it's not needed. (Is documentation required?) - [-] I have properly separated EE content from FOSS, or this MR is FOSS only. (Where should EE code go?)
- [-] I have added information for database reviewers in the MR description, or it's not needed. (Does this MR have database related changes?)
-
I have self-reviewed this MR per code review guidelines. - [-] This MR does not harm performance, or I have asked a reviewer to help assess the performance impact. (Merge request performance guidelines)
-
I have followed the style guides. -
This change is backwards compatible across updates, or this does not apply.
Merge request reports
Activity
assigned to @dzaporozhets
changed milestone to %14.3
added backend devopsmonitor grouprespond labels
marked the checklist item I have included changelog trailers, or none are needed. (Does this MR need a changelog?) as completed
marked the checklist item I have added/updated documentation, or it's not needed. (Is documentation required?) as completed
- A deleted user
added documentation label
1 Message This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. Documentation review
The following files require a review from a technical writer:
doc/operations/error_tracking.md
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
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 Serena Fang ( @serenafang
) (UTC-5, 8 hours behind@dzaporozhets
)Charlie Ablett ( @cablett
) (UTC+12, 9 hours ahead of@dzaporozhets
)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
Dangeradded 1 commit
- a1a2aaa9 - Remove integrated_error_tracking feature flag
mentioned in issue #335846 (closed)
added sectionops label
- Resolved by Matthias Käppler
@eurie please review
requested review from @eurie
@eurie
, thanks for approving this merge request.This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.
For more info, please refer to the following links:
requested review from @mkaeppler and removed review request for @eurie
added 554 commits
-
a1a2aaa9...25697fb4 - 553 commits from branch
master
- f9f246c6 - Remove integrated_error_tracking feature flag
-
a1a2aaa9...25697fb4 - 553 commits from branch
added typemaintenance label
added typefeature label
enabled an automatic merge when the pipeline for bfc83482 succeeds
mentioned in commit adf91a36
added workflowstaging label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
added workflowstaging-canary label and removed workflowproduction label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label
removed typefeature label
added Category:Error Tracking label
mentioned in merge request !81767 (merged)