diff options
| author | Giuseppe Scrivano <gscrivan@redhat.com> | 2019-05-21 10:01:29 +0200 |
|---|---|---|
| committer | Giuseppe Scrivano <gscrivan@redhat.com> | 2019-05-21 10:07:31 +0200 |
| commit | 3788da93445dffda35a00366c90a9d8da6ef0702 (patch) | |
| tree | 97d34259794a142b93e046a32184158029211cf7 /libpod/container_internal.go | |
| parent | e43a98e512603d8608144efbcd3e9dde5759823f (diff) | |
| download | podman-3788da93445dffda35a00366c90a9d8da6ef0702.tar.gz podman-3788da93445dffda35a00366c90a9d8da6ef0702.tar.bz2 podman-3788da93445dffda35a00366c90a9d8da6ef0702.zip | |
libpod: prefer WaitForFile to polling
replace two usage of kwait.ExponentialBackoff in favor of WaitForFile
that uses inotify when possible.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Diffstat (limited to 'libpod/container_internal.go')
| -rw-r--r-- | libpod/container_internal.go | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/libpod/container_internal.go b/libpod/container_internal.go index fc33a1bbc..5f8dd1c72 100644 --- a/libpod/container_internal.go +++ b/libpod/container_internal.go @@ -25,7 +25,6 @@ import ( opentracing "github.com/opentracing/opentracing-go" "github.com/pkg/errors" "github.com/sirupsen/logrus" - kwait "k8s.io/apimachinery/pkg/util/wait" ) const ( @@ -146,20 +145,10 @@ func (c *Container) exitFilePath() string { func (c *Container) waitForExitFileAndSync() error { exitFile := c.exitFilePath() - err := kwait.ExponentialBackoff( - kwait.Backoff{ - Duration: 500 * time.Millisecond, - Factor: 1.2, - Steps: 6, - }, - func() (bool, error) { - _, err := os.Stat(exitFile) - if err != nil { - // wait longer - return false, nil - } - return true, nil - }) + chWait := make(chan error) + defer close(chWait) + + _, err := WaitForFile(exitFile, chWait, time.Second*5) if err != nil { // Exit file did not appear // Reset our state |