Commit 5f0288e0 authored by Random-Liu's avatar Random-Liu

Double `StopContainer` request timeout.

parent b6211c6e
...@@ -209,11 +209,10 @@ func (r *RemoteRuntimeService) StartContainer(containerID string) error { ...@@ -209,11 +209,10 @@ func (r *RemoteRuntimeService) StartContainer(containerID string) error {
// StopContainer stops a running container with a grace period (i.e., timeout). // StopContainer stops a running container with a grace period (i.e., timeout).
func (r *RemoteRuntimeService) StopContainer(containerID string, timeout int64) error { func (r *RemoteRuntimeService) StopContainer(containerID string, timeout int64) error {
ctx, cancel := getContextWithTimeout(time.Duration(timeout) * time.Second) // Use timeout + default timeout (2 minutes) as timeout to leave extra time
if timeout == 0 { // for SIGKILL container and request latency.
// Use default timeout if stop timeout is 0. t := r.timeout + time.Duration(timeout)*time.Second
ctx, cancel = getContextWithTimeout(r.timeout) ctx, cancel := getContextWithTimeout(t)
}
defer cancel() defer cancel()
_, err := r.runtimeClient.StopContainer(ctx, &runtimeapi.StopContainerRequest{ _, err := r.runtimeClient.StopContainer(ctx, &runtimeapi.StopContainerRequest{
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment