github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-05 10:19:50 +00:00
Code Issues Projects Releases Wiki Activity

Don't leak tickers in the exec probe

Browse Source
This commit is contained in:
Prashanth Balasubramanian 2015-07-05 17:03:10 -07:00
parent bb6f2f7ad9
commit 33e2f7cb8b
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/kubelet/dockertools/exec.go
View File

@ -123,7 +123,8 @@ func (*NativeExecHandler) ExecInContainer(client DockerInterface, container *doc
if err != nil {
return err
}
tick := time.Tick(2 * time.Second)
ticker := time.NewTicker(2 * time.Second)
defer ticker.Stop()
count := 0
for {
inspect, err2 := client.InspectExec(execObj.ID)
@ -143,7 +144,7 @@ func (*NativeExecHandler) ExecInContainer(client DockerInterface, container *doc
break
}
<-tick
<-ticker.C
}
return err

5
pkg/kubelet/dockertools/manager.go
View File

@ -945,7 +945,8 @@ func (dm *DockerManager) RunInContainer(containerID string, cmd []string) ([]byt
glog.V(2).Infof("StartExec With error: %v", err)
return nil, err
}
tick := time.Tick(2 * time.Second)
ticker := time.NewTicker(2 * time.Second)
defer ticker.Stop()
for {
inspect, err2 := dm.client.InspectExec(execObj.ID)
if err2 != nil {
@ -959,7 +960,7 @@ func (dm *DockerManager) RunInContainer(containerID string, cmd []string) ([]byt
}
break
}
<-tick
<-ticker.C
}
return buf.Bytes(), err