Code review comment for lp://staging/~doanac/ubuntu-ci-services-itself/source-builder-design

Revision history for this message
Evan (ev) wrote :

Checking rabbit queues is a solved problem :)

https://bazaar.launchpad.net/~canonical-is-robot/canonical-is-puppet/golive/view/head:/modules/rabbitmq/files/check_rabbitmq.py
https://bazaar.launchpad.net/~canonical-is-robot/canonical-is-puppet/golive/view/head:/modules/rabbitmq/files/check_rabbitmq_queues.py

That's run as a standard nrpe (Nagios) check. I suspect we want to
isolate that check to be directly run off the Rabbit server, rather
than going through one of our other services. If you have a more
specific check for the Source Builder, then I'd agree that putting
that behind the heath_check API call makes a lot of sense.

« Back to merge proposal