raft: Skip transport tests if Praefect is enabled
This MR skips Raft transport tests if Praefect is enabled. This combination cause severe flakiness in Gitaly test suite so that it takes a dozen retry attempts to merge an MR. For example: https://gitlab.com/gitlab-org/gitaly/-/jobs/9180117817.
Merge request reports
Activity
changed milestone to %17.10
added groupgitaly maintenancetest-gap typemaintenance labels
assigned to @qmnguyen0711
Reviewer roulette
Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot. 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, mention them as you normally would! Danger does not automatically notify them for you.
Category Reviewer Maintainer None @divya_gitlab
(UTC+5.5, 1.5 hours behind author)
@echui-gitlab
(UTC+11, 4 hours ahead of author)
If needed, you can retry the
🔁 danger-review
job that generated this comment.Generated by
🚫 DangerEdited by ****@divya_gitlab As discussed in Slack, I excluded transport tests when Praefect is enabled. Could you have a look?
requested review from @divya_gitlab
started a merge train
mentioned in commit 0c8e0cb5
mentioned in merge request !7626 (merged)
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