Merge pull request #8206 from kaovilai/pkgpodvolumebackupper_test.go59714thecancelfunctionreturnedbycontext.WithTimeout
pkg/podvolume/backupper_test.go:597:14: the cancel function returned by context.WithTimeout should be called, not discarded, to avoid a context leakpull/8208/head
commit
7e8a3c0bbc
|
@ -594,7 +594,10 @@ func (l *logHook) Fire(entry *logrus.Entry) error {
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestWaitAllPodVolumesProcessed(t *testing.T) {
|
func TestWaitAllPodVolumesProcessed(t *testing.T) {
|
||||||
timeoutCtx, _ := context.WithTimeout(context.Background(), 1*time.Second)
|
timeoutCtx, cancelFunc := context.WithTimeout(context.Background(), 1*time.Second)
|
||||||
|
defer func() {
|
||||||
|
cancelFunc()
|
||||||
|
}()
|
||||||
cases := []struct {
|
cases := []struct {
|
||||||
name string
|
name string
|
||||||
ctx context.Context
|
ctx context.Context
|
||||||
|
|
Loading…
Reference in New Issue