-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(actor) Remove old actor columns #70549
Conversation
This PR has a migration; here is the generated SQL for --
-- Custom state/database change combination
--
ALTER TABLE sentry_team DROP COLUMN actor_id;
ALTER TABLE sentry_rule DROP COLUMN owner_id;
ALTER TABLE sentry_alertrule DROP COLUMN owner_id;
ALTER TABLE sentry_grouphistory DROP COLUMN actor_id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #70549 +/- ##
==========================================
+ Coverage 76.99% 79.98% +2.98%
==========================================
Files 6496 6498 +2
Lines 290058 290232 +174
Branches 49979 50008 +29
==========================================
+ Hits 223340 232129 +8789
+ Misses 66349 57736 -8613
+ Partials 369 367 -2 |
PR reverted: cb038e9 |
This reverts commit 1e5349e. Co-authored-by: markstory <24086+markstory@users.noreply.github.com>
Remove now unused actor columns from rule, alertrule, team, grouphistory. These columns were removed from django state in #69784, #69873, #69976, #70090
Refs HC-1178