Code review comment for lp:~doanac/ubuntu-ci-services-itself/rabbit-queue-status

Revision history for this message
Vincent Ladeuil (vila) wrote :

62 + '''checks if there any workers attached to the queue.'''

'are' missing ?

66 + # we already report rabbit isn't configured, so no sense adding
67 + # another failure for this

Let's have a common way to report that error then, in the most obscure cases, it's better to have *some* output than none. I know this will never happen... but I know that if it happens, we'll be glad to have an error message ;)

review: Approve

« Back to merge proposal