Code review comment for lp:~qzhang/lava-scheduler/retrieve-job-fail-reason

Revision history for this message
Paul Larson (pwlars) wrote :

I don't think we want to do this. One of the uses of health checks is to catch problems with lava after performing an update. The idea is that we would set all health statuses to unknown when doing the update, and force a health check before they process any new jobs. If there's a bug we introduced that drastically interferes with processing jobs then it can catch it, and prevent any more jobs from running.

review: Disapprove

« Back to merge proposal