Send a weekly instance status report (resolves #1509) #1683
Conversation
Twixes
changed the title
posthog/celery.py
Outdated
| @@ -48,6 +47,9 @@ def setup_periodic_tasks(sender, **kwargs): | |||
| sender.add_periodic_task( | |||
| crontab(day_of_week="mon,fri"), update_event_partitions.s(), # check twice a week | |||
| ) | |||
| sender.add_periodic_task( | |||
| 10, status_report.s(), | |||
There was a problem hiding this comment.
This would send the report every 10 seconds, perhaps a cron schedule here? Also, what should we do with retries or if the server is down during the scheduled time (e.g. Heroku free dynos). Ideally we would also have a failsafe on PostHog side to prevent duplicate entries for the same week.
There was a problem hiding this comment.
Related to the above, we might have the edge case of a user having multiple instances or workers, so event duplication prevention might become more relevant (doing reports on a team-basis would also help here).
There was a problem hiding this comment.
Ah, sorry, this is a leftover from testing, supposed to be crontab(day_of_week="mon"). :)
|
A-ha, there is one odd issue: |
|
This now reports correctly to PostHog. Also added the count of persons active in the period.
|
Changes
This adds a weekly (sent to us each Monday) status report regarding each PostHog instance as a whole, including event ingestion metadata.