Skip to content

Conversation

@crankynetman
Copy link
Collaborator

@crankynetman crankynetman commented Jan 27, 2026

Just because using docker healthchecks for this bothers me.

Plus, now it's easy to disable process_updates for testing and we have some form of record of the responses.

@github-actions
Copy link

Pytest Scheduler Coverage

File Coverage
All files 100%

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against f953ffb

@github-actions
Copy link

github-actions bot commented Jan 27, 2026

Django Pytest/Behave Coverage

File Coverage
All files 83%
config/consumers.py 78%
config/urls.py 69%
config/settings/base.py 69%
config/settings/local.py 72%
scram/route_manager/admin.py 71%
scram/route_manager/models.py 79%
scram/route_manager/views.py 88%
scram/route_manager/api/serializers.py 92%
scram/route_manager/api/views.py 76%
scram/shared/shared_code.py 56%
scram/templates/403.html 91%
scram/templates/404.html 91%
scram/templates/base.html 99%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 53064e7

@github-actions
Copy link

github-actions bot commented Jan 27, 2026

Pytest Celery Scheduler Coverage

File Coverage
All files 100%

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against 53064e7

@crankynetman crankynetman changed the title Topic/chriscummings/celery health check replacement chore(process_updates): move away from docker check to celery Jan 27, 2026
Copy link
Collaborator

@samoehlert samoehlert left a comment

Choose a reason for hiding this comment

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

Big win looks great!

@crankynetman crankynetman enabled auto-merge (squash) January 28, 2026 19:17
@crankynetman crankynetman merged commit 8a7f6d9 into main Jan 28, 2026
18 of 20 checks passed
@crankynetman crankynetman deleted the topic/chriscummings/celery-health-check-replacement branch January 28, 2026 19:19
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.

3 participants