Skip to content

Add celery beat task to publish queue depth to CloudWatch - #1196

Draft
ericholscher wants to merge 1 commit into
mainfrom
celery-queue-depth-metric
Draft

Add celery beat task to publish queue depth to CloudWatch#1196
ericholscher wants to merge 1 commit into
mainfrom
celery-queue-depth-metric

Conversation

@ericholscher

@ericholscher ericholscher commented Apr 7, 2026

Copy link
Copy Markdown
Member

Summary

  • Registers ethicalads_ext.monitoring in INSTALLED_APPS (conditional on ADSERVER_EXT)
  • Adds beat schedule entry for publish_celery_queue_depth task (every minute)
  • Task implementation lives in ethicalads-ext (readthedocs/ethicalads-ext companion PR)
  • Used by the celery ASG autoscaling policy in readthedocs/ethicalads-ops#64

Test plan

  • Verify task is discovered and runs when ADSERVER_EXT is set
  • Verify beat schedule entry does not appear without ADSERVER_EXT

Register the monitoring module and add a beat schedule entry for the
queue depth CloudWatch metric task (in ethicalads-ext).
@ericholscher
ericholscher force-pushed the celery-queue-depth-metric branch from 300f4e7 to 88161ff Compare April 7, 2026 01:18

@davidfischer davidfischer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks OK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants