Merge pull request #83376 from liyanhui1228/fix_test

Fix sig-node test by adding back the numNodes
kubernetes · Oct 3, 2019 · 8e39c60 · 8e39c60
2 parents ae944c6 + c06413b
commit 8e39c60
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/test/e2e/node/kubelet.go b/test/e2e/node/kubelet.go
@@ -272,6 +272,7 @@ var _ = SIGDescribe("kubelet", func() {
 			nodeLabels = make(map[string]string)
 			nodeLabels["kubelet_cleanup"] = "true"
 			nodes, err := e2enode.GetBoundedReadySchedulableNodes(c, maxNodesToCheck)
+			numNodes = len(nodes.Items)
 			framework.ExpectNoError(err)
 			nodeNames = sets.NewString()
 			for i := 0; i < len(nodes.Items); i++ {
@@ -284,7 +285,7 @@ var _ = SIGDescribe("kubelet", func() {
 			}
 
 			// Start resourceMonitor only in small clusters.
-			if len(nodes.Items) <= maxNodesToCheck {
+			if numNodes <= maxNodesToCheck {
 				resourceMonitor = e2ekubelet.NewResourceMonitor(f.ClientSet, e2ekubelet.TargetContainers(), containerStatsPollingInterval)
 				resourceMonitor.Start()
 			}