diff options
author | Daniel J Walsh <dwalsh@redhat.com> | 2021-09-10 11:19:59 -0400 |
---|---|---|
committer | Daniel J Walsh <dwalsh@redhat.com> | 2021-09-10 11:20:01 -0400 |
commit | 4fbc5b8fe75b3d486a97c013af7a6b085292b46a (patch) | |
tree | a31f855d21de2c3837821a1fa4c7fb386672c177 /pkg/bindings | |
parent | e6046224ea88cad9286303456562b4a24ad9cf9b (diff) | |
download | podman-4fbc5b8fe75b3d486a97c013af7a6b085292b46a.tar.gz podman-4fbc5b8fe75b3d486a97c013af7a6b085292b46a.tar.bz2 podman-4fbc5b8fe75b3d486a97c013af7a6b085292b46a.zip |
Stop outputting 'healthy' on healthcheck
We should only print unhealthy if the check fails. Currently this is
filling logs when users are running lots of healthchecks.
Improves: https://github.com/containers/podman/issues/11157
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/bindings')
-rw-r--r-- | pkg/bindings/test/containers_test.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/pkg/bindings/test/containers_test.go b/pkg/bindings/test/containers_test.go index 4d1361746..b9ed67255 100644 --- a/pkg/bindings/test/containers_test.go +++ b/pkg/bindings/test/containers_test.go @@ -326,11 +326,11 @@ var _ = Describe("Podman containers ", func() { // TODO for the life of me, i cannot get this to work. maybe another set // of eyes will // successful healthcheck - //status := "healthy" + //status := define.HealthCheckHealthy //for i:=0; i < 10; i++ { // result, err := containers.RunHealthCheck(connText, "hc") // Expect(err).To(BeNil()) - // if result.Status != "healthy" { + // if result.Status != define.HealthCheckHealthy { // fmt.Println("Healthcheck container still starting, retrying in 1 second") // time.Sleep(1 * time.Second) // continue @@ -338,7 +338,7 @@ var _ = Describe("Podman containers ", func() { // status = result.Status // break //} - //Expect(status).To(Equal("healthy")) + //Expect(status).To(Equal(define.HealthCheckHealthy)) // TODO enable this when wait is working // healthcheck on a stopped container should be a 409 |