diff options
author | Matthew Heon <matthew.heon@pm.me> | 2021-01-08 11:24:54 -0500 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2021-01-11 09:30:35 -0500 |
commit | 7e3fb33be85122d509756e197aac10c4cf9930b6 (patch) | |
tree | 4f39629c5d9191befa2f718b0e2596f805bdb26f /pkg/domain/infra/abi/healthcheck.go | |
parent | 49db79e735acd2c693762eaff62680cd9a8cb60b (diff) | |
download | podman-7e3fb33be85122d509756e197aac10c4cf9930b6.tar.gz podman-7e3fb33be85122d509756e197aac10c4cf9930b6.tar.bz2 podman-7e3fb33be85122d509756e197aac10c4cf9930b6.zip |
Ensure that `podman play kube` actually reports errors
In 2.2.x, we moved `play kube` to use the Start() API for pods,
which reported errors in a different way (all containers are
started in parallel, and then results reported as a block). The
migration attempted to preserve compatibility by returning only
one error, but that's not really a viable option as it can
obscure the real reason that a pod is failing. Further, the code
was not correctly handling the API's errors - Pod Start() will,
on any container error, return a map of container ID to error
populated for all container errors *and* return ErrPodPartialFail
for overall error - the existing code did not handle the partial
failure error and thus would never return container errors.
Refactor the `play kube` API to include a set of errors for
containers in each pod, so we can return all errors that occurred
to the frontend and print them for the user, and correct the
backend code so container errors are actually forwarded.
Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'pkg/domain/infra/abi/healthcheck.go')
0 files changed, 0 insertions, 0 deletions