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