shiplift icon indicating copy to clipboard operation
shiplift copied to clipboard

Add healthcheck-related fields to the container`s state

Open andrey-yantsen opened this issue 3 years ago • 0 comments

What did you implement:

This PR adds support for deserializing of the Healthcheck fields in the docker inspect results (see API for the details).

I'm not sure about HealthStatus::Unknown. Still, I decided to add it to prevent panic when deserializing a response from a hypothetical future docker version, where new health statuses would be added.

How did you verify your change:

I've tried deserializing the docker inspect response, produced by a container with (and without) a health check. Also, I tried the deserialization with/without the chrono feature enabled.

andrey-yantsen avatar Jan 13 '22 20:01 andrey-yantsen