From da85510bd3329809507a3272d5394d8bd1fc8ede Mon Sep 17 00:00:00 2001 From: Kenichi Omichi Date: Wed, 28 Aug 2019 23:08:44 +0000 Subject: [PATCH] Reduce redundant Nodes().List() call dumpAllNodeInfo() called Nodes().List() internally, but the function is called from DumpAllNamespaceInfo() only and DumpAllNamespaceInfo() calls Nodes().List() before calling dumpAllNodeInfo(). So this makes the result of Nodes().List() being passed to dumpAllNodeInfo() then reduce a call of Nodes().List(). --- test/e2e/framework/util.go | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/test/e2e/framework/util.go b/test/e2e/framework/util.go index 356b41cf591..a84be9a4ac3 100644 --- a/test/e2e/framework/util.go +++ b/test/e2e/framework/util.go @@ -1775,7 +1775,7 @@ func DumpAllNamespaceInfo(c clientset.Interface, namespace string) { return } if len(nodes.Items) <= maxNodesForDump { - dumpAllNodeInfo(c) + dumpAllNodeInfo(c, nodes) } else { e2elog.Logf("skipping dumping cluster info - cluster too large") } @@ -1794,13 +1794,7 @@ func (o byFirstTimestamp) Less(i, j int) bool { return o[i].FirstTimestamp.Before(&o[j].FirstTimestamp) } -func dumpAllNodeInfo(c clientset.Interface) { - // It should be OK to list unschedulable Nodes here. - nodes, err := c.CoreV1().Nodes().List(metav1.ListOptions{}) - if err != nil { - e2elog.Logf("unable to fetch node list: %v", err) - return - } +func dumpAllNodeInfo(c clientset.Interface, nodes *v1.NodeList) { names := make([]string, len(nodes.Items)) for ix := range nodes.Items { names[ix] = nodes.Items[ix].Name