Merge pull request #26642 from wojtek-t/fix_kubelet_test_panics

Fix panic in kubelet test
pull/6/head
Alex Robinson 2016-06-01 15:41:35 -04:00
commit 903821354e
1 changed files with 7 additions and 3 deletions

View File

@ -155,7 +155,7 @@ var _ = framework.KubeDescribe("kubelet", func() {
updateNodeLabels(c, nodeNames, nodeLabels, nil)
// Start resourceMonitor only in small clusters.
if len(nodes.Items) < maxNodesToCheck {
if len(nodes.Items) <= maxNodesToCheck {
resourceMonitor = framework.NewResourceMonitor(f.Client, framework.TargetContainers(), containerStatsPollingInterval)
resourceMonitor.Start()
}
@ -199,7 +199,9 @@ var _ = framework.KubeDescribe("kubelet", func() {
// transition to the running status.
Expect(waitTillNPodsRunningOnNodes(f.Client, nodeNames, rcName, f.Namespace.Name, totalPods,
time.Second*30)).NotTo(HaveOccurred())
if resourceMonitor != nil {
resourceMonitor.LogLatest()
}
By("Deleting the RC")
framework.DeleteRC(f.Client, f.Namespace.Name, rcName)
@ -215,7 +217,9 @@ var _ = framework.KubeDescribe("kubelet", func() {
itArg.timeout)).NotTo(HaveOccurred())
framework.Logf("Deleting %d pods on %d nodes completed in %v after the RC was deleted", totalPods, len(nodeNames),
time.Since(start))
if resourceMonitor != nil {
resourceMonitor.LogCPUSummary()
}
})
}
})