Add Sentry Issue Finder
What does this MR do?
This adds a SentryIssueFinder
, which will be used in a subsequent MR in order to find Issues that are related to a Sentry Error & close them (part of #39825 (closed)).
This provides the ability to find SentryIssue
by their sentry_issue_identifier
.
Does this MR meet the acceptance criteria?
Conformity
- [N/A] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Related to #39825 (closed)
Merge request reports
Activity
changed milestone to %12.7
removed typefeature label
removed documentation frontend release post item labels
removed direction label
1 Error This merge request includes more than 10 commits. Please rebase these commits into a smaller number of commits. 1 Message This merge request adds or changes files that require a review from the Database team. This merge request requires a database review. To make sure these changes are reviewed, take the following steps:
- Ensure the merge request has database and databasereview pending labels. If the merge request modifies database files, Danger will do this for you.
- Prepare your MR for database review according to the docs.
- Assign and mention the database reviewer suggested by Reviewer Roulette.
The following files require a review from the Database team:
db/migrate/20200114204949_add_index_to_sentry_issues_sentry_issue_identifier.rb
db/schema.rb
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 (e.g. frontend or backend), and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override this selection if you think someone else would be better-suited, or the chosen person is unavailable.
Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.
Category Reviewer Maintainer backend Markus Koller ( @toupeira
)Rémy Coutable ( @rymai
)database Krasimir Angelov ( @krasio
)Toon Claes ( @toon
)Generated by
DangerEdited by 🤖 GitLab Bot 🤖- Resolved by Nick Thomas
- Resolved by Sarah Yasonik
Looking good! Nothing major from me @seanarnold.
assigned to @syasonik
mentioned in merge request !22744 (merged)
- Resolved by Nick Thomas
- Resolved by Nick Thomas
- Resolved by Nick Thomas
- Resolved by Nick Thomas
- Resolved by Nick Thomas
- Resolved by Nick Thomas