Skip to content

feat(async): Add django-q2 with ORM broker and demo job#161

Open
taylor-osler-sentry wants to merge 3 commits intomainfrom
feat/async
Open

feat(async): Add django-q2 with ORM broker and demo job#161
taylor-osler-sentry wants to merge 3 commits intomainfrom
feat/async

Conversation

@taylor-osler-sentry
Copy link
Copy Markdown
Contributor

@taylor-osler-sentry taylor-osler-sentry commented Apr 30, 2026

Summary

  • Adds django-q2 as a dependency and configures it with the ORM broker (backed by the existing PostgreSQL default database)
  • Introduces firetower.incidents.tasks.schedule_demo as the first queue task (stub that prints "hello world"), registered via a data migration
  • Extends docker/entrypoint.sh with a worker command that starts qcluster, and refactors all branches to use exec for proper signal handling in Docker

Test plan

  • Run python manage.py migrate — confirm 0015_schedule_demo applies cleanly and a schedule_slo_nag row appears in django_q_schedule
  • Run python manage.py qcluster — confirm the worker starts and picks up the scheduled task
  • Run docker/entrypoint.sh worker — confirm the container starts the qcluster process
  • Unapply migration (migrate incidents 0014) — confirm the schedule row is removed

🤖 Generated with Claude Code

@taylor-osler-sentry taylor-osler-sentry requested a review from a team as a code owner April 30, 2026 21:53
@taylor-osler-sentry taylor-osler-sentry marked this pull request as draft April 30, 2026 21:53
Comment thread src/firetower/settings.py Outdated
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 30, 2026

Comment thread src/firetower/incidents/apps.py Outdated
Comment thread src/firetower/incidents/tasks.py
Comment thread src/firetower/incidents/tasks.py
Comment thread src/firetower/settings.py
Comment thread src/firetower/settings.py Outdated
@taylor-osler-sentry taylor-osler-sentry changed the title feat(async): Add django-q2 with ORM broker and schedule_slo_nag job feat(async): Add django-q2 with ORM broker and demo job May 5, 2026
@taylor-osler-sentry taylor-osler-sentry marked this pull request as ready for review May 5, 2026 21:43
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.

1 participant