Improve test clean up for migration spec

changed milestone to %16.4

added bugtransient groupdatabase typebug labels

assigned to @krasio

added devopsdata stores sectioncore platform labels

mentioned in issue gitlab-org/quality/engineering-productivity/master-broken-incidents#3341 (closed)

added master:broken pipeline:expedite labels

added database databasereview pending labels

1 Message

CHANGELOG missing:

If you want to create a changelog entry for GitLab FOSS, add the Changelog trailer to the commit message you want to add to the changelog.

If you want to create a changelog entry for GitLab EE, also add the EE: true trailer to your commit message.

If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

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
database	Tianwen Chen (`@tianwenchen`) (UTC+8, 5 hours ahead of `@krasio`)	Marius Bobin (`@mbobin`) (UTC+3, same timezone as `@krasio`)

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 Danger

changed the description

approved this merge request

added databaseapproved label and removed databasereview pending label

enabled an automatic merge when the pipeline for b0d655b3 succeeds

aborted the automatic merge because source branch was updated

added 1 commit

30449e6b - Improve test clean up

Compare with previous version

changed the description

requested review from @mbobin

approved this merge request

enabled an automatic merge when the pipeline for 3dfe86cd succeeds

marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

merged

Hello @krasio

The database team is looking for ways to improve the database review process and we would love your help!

If you'd be open to someone on the database team reaching out to you for a chat, or if you'd like to leave some feedback asynchronously, just post a reply to this comment mentioning: