-
-
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
feat(tempest): Add cron job and tasks #82454
feat(tempest): Add cron job and tasks #82454
Conversation
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #82454 +/- ##
==========================================
- Coverage 87.53% 87.50% -0.03%
==========================================
Files 9481 9403 -78
Lines 537762 537550 -212
Branches 21178 21178
==========================================
- Hits 470711 470393 -318
- Misses 66701 66807 +106
Partials 350 350 |
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.
LGTM - as discussed, please have @vgrozdanic sign this off to make sure his comments have been addressed.
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.
🚀
Few hours ago #82454 was deployed, but there we have missed to register celery tasks which cause another error: https://sentry.sentry.io/issues/4271076192/?project=1&query=is%3Aunresolved%20sentry.tempest.tasks.poll_tempest&referrer=issue-stream&sort=date&statsPeriod=2h&stream_index=0 This PR registers the celery tasks and should also fix the Sentry error
Few hours ago #82454 was deployed, but there we have missed to register celery tasks which cause another error: https://sentry.sentry.io/issues/4271076192/?project=1&query=is%3Aunresolved%20sentry.tempest.tasks.poll_tempest&referrer=issue-stream&sort=date&statsPeriod=2h&stream_index=0 This PR registers the celery tasks and should also fix the Sentry error
Fixes: https://github.com/getsentry/team-gdx/issues/31