Re: Request for Comments - Health Check API Proposal

2018-10-17 Thread Vinod Kone
One of the things we discussed when we added `CheckInfo` and `CheckStatusInfo` was to make the older `HealthCheck` and `bool healthy` field (inside `TaskStatus`) consistent with the new `Check` format. IIRC, some of the changes we wanted to do were - Deprecate `HealthCheck` and introduce a

Request for Comments - Health Check API Proposal

2018-10-17 Thread Greg Mann
Hi all, Some users have recently reported issues with our current implementation of health checks. See this ticket for an introduction to the issue. To summarize: we currently use a single 'optional bool healthy' field within the 'TaskStatus'