From a44e335940f6df5fa3259846a334447faf17db2a Mon Sep 17 00:00:00 2001 From: Shyam Jeedigunta Date: Tue, 3 Apr 2018 17:37:49 +0200 Subject: [PATCH] Don't quit without printing API latencies in density test if it failed --- test/e2e/scalability/density.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/e2e/scalability/density.go b/test/e2e/scalability/density.go index 8476e26137f..7502f2edf3f 100644 --- a/test/e2e/scalability/density.go +++ b/test/e2e/scalability/density.go @@ -386,7 +386,6 @@ var _ = SIGDescribe("Density", func() { framework.ExpectNoError(err) if err == nil { summaries = append(summaries, metrics) - Expect(highLatencyRequests).NotTo(BeNumerically(">", 0), "There should be no high-latency requests") } // Verify scheduler metrics. @@ -401,6 +400,8 @@ var _ = SIGDescribe("Density", func() { framework.PrintSummaries(summaries, testCaseBaseName) + // Fail if there were some high-latency requests. + Expect(highLatencyRequests).NotTo(BeNumerically(">", 0), "There should be no high-latency requests") // Fail if more than the allowed threshold of measurements were missing in the latencyTest. Expect(missingMeasurements <= MaxMissingPodStartupMeasurements).To(Equal(true)) })