-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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(hybridcloud) Move outbox tasks to sentry.hybridcloud #75739
Conversation
Consolidate more outbox subsystem logic into sentry.hybridcloud.
"sentry.hybridcloud.tasks.backfill_outboxes", | ||
"sentry.hybridcloud.tasks.deliver_from_outbox", |
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.
The tasks didn't have their names changed, so updating CELERY_IMPORTS
should be all that is necessary for tasks to continue executing without interruption.
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #75739 +/- ##
==========================================
- Coverage 78.25% 78.24% -0.01%
==========================================
Files 6838 6815 -23
Lines 303803 303669 -134
Branches 52271 52232 -39
==========================================
- Hits 237730 237604 -126
+ Misses 59693 59681 -12
- Partials 6380 6384 +4
|
@@ -78,7 +80,7 @@ def test_region_processing(task_runner): | |||
|
|||
@django_db_all | |||
@control_silo_test | |||
def test_control_processing(task_runner): | |||
def test_control_processing(task_runner: Callable[..., Any]) -> None: |
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.
Do we have any type exemptions on these files that we need to clear now that we're explicitly typing things?
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.
In their new location they get opted into our standard mypy rules which led to additional types being added.
Consolidate more outbox subsystem logic into sentry.hybridcloud.