Increase pidWaitTimeout to 60s

At scale, it appears that we sometimes hit the 1000ms timeout to create
the PID file when a container is created or executed.
Increasing the value to 60s should help when running a lot of containers
in heavy-loaded environment.

Related #1495
Fixes #1816
Signed-off-by: Emilien Macchi <emilien@redhat.com>
This commit is contained in:
Emilien Macchi
2018-11-15 09:32:51 -05:00
parent 97bded4a52
commit c3d8328150

View File

@ -330,9 +330,10 @@ func (c *Container) Exec(tty, privileged bool, env, cmd []string, user string) e
} }
pidFile := c.execPidPath(sessionID) pidFile := c.execPidPath(sessionID)
// 1 second seems a reasonable time to wait // 60 second seems a reasonable time to wait
// See https://github.com/containers/libpod/issues/1495 // https://github.com/containers/libpod/issues/1495
const pidWaitTimeout = 1000 // https://github.com/containers/libpod/issues/1816
const pidWaitTimeout = 60000
// Wait until the runtime makes the pidfile // Wait until the runtime makes the pidfile
// TODO: If runtime errors before the PID file is created, we have to // TODO: If runtime errors before the PID file is created, we have to