Skip to content
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

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

markstory
Copy link
Member

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

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
@markstory markstory requested a review from a team as a code owner May 8, 2024 21:24
@markstory markstory requested a review from a team May 8, 2024 21:24
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 8, 2024
Copy link
Contributor

github-actions bot commented May 8, 2024

This PR has a migration; here is the generated SQL for src/sentry/migrations/0718_remove_actor_columns.py ()

--
-- 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;

Copy link
Member

@wedamija wedamija left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

Copy link

codecov bot commented May 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.98%. Comparing base (9fc6f1d) to head (94d5f87).
Report is 9 commits behind head on master.

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     

see 630 files with indirect coverage changes

@markstory markstory merged commit 1e5349e into master May 9, 2024
50 checks passed
@markstory markstory deleted the chore-remove-actor-columns branch May 9, 2024 14:04
@markstory markstory added the Trigger: Revert add to a merged PR to revert it (skips CI) label May 9, 2024
@getsentry-bot
Copy link
Contributor

PR reverted: cb038e9

getsentry-bot added a commit that referenced this pull request May 9, 2024
This reverts commit 1e5349e.

Co-authored-by: markstory <24086+markstory@users.noreply.github.com>
markstory added a commit that referenced this pull request May 9, 2024
markstory added a commit that referenced this pull request May 10, 2024
Restore changes from #70549 which had to be reverted as they caused
failures in getsentry migration tests.

This reverts commit cb038e9.
@github-actions github-actions bot locked and limited conversation to collaborators May 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert add to a merged PR to revert it (skips CI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants