Skip to content

K
k3s
Commit a44e3359 authored by Shyam Jeedigunta's avatar Shyam Jeedigunta
Browse files
Options

Don't quit without printing API latencies in density test if it failed

parent a5133305
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 1 deletion
test/e2e/scalability/density.go
View file @ a44e3359
...@@ -386,7 +386,6 @@ var _ = SIGDescribe("Density", func() { ...@@ -386,7 +386,6 @@ var _ = SIGDescribe("Density", func() {
framework.ExpectNoError(err) framework.ExpectNoError(err)
if err == nil { if err == nil {
summaries = append(summaries, metrics) summaries = append(summaries, metrics)
Expect(highLatencyRequests).NotTo(BeNumerically(">", 0), "There should be no high-latency requests")
} }
// Verify scheduler metrics. // Verify scheduler metrics.
...@@ -401,6 +400,8 @@ var _ = SIGDescribe("Density", func() { ...@@ -401,6 +400,8 @@ var _ = SIGDescribe("Density", func() {
framework.PrintSummaries(summaries, testCaseBaseName) 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. // Fail if more than the allowed threshold of measurements were missing in the latencyTest.
Expect(missingMeasurements <= MaxMissingPodStartupMeasurements).To(Equal(true)) Expect(missingMeasurements <= MaxMissingPodStartupMeasurements).To(Equal(true))
}) })
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment