From 9f6702afbc5f6dacb377dafe89c522dbdb701880 Mon Sep 17 00:00:00 2001 From: Benjamin Bennett Date: Wed, 26 Jul 2017 16:35:56 -0400 Subject: [PATCH] Correct the url for the memory stats in the router test We were missing the debug=1 argument on the memory stats, so were getting them in a binary format. With the flag added we pass tests properly. Fixes https://github.com/openshift/origin/issues/15356 --- test/extended/router/metrics.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/extended/router/metrics.go b/test/extended/router/metrics.go index 9fc11863fbad..630dd026064b 100644 --- a/test/extended/router/metrics.go +++ b/test/extended/router/metrics.go @@ -178,7 +178,7 @@ var _ = g.Describe("[Conformance][networking][router] openshift router metrics", o.Expect(err).NotTo(o.HaveOccurred()) g.By("at /debug/pprof") - results, err := getAuthenticatedURLViaPod(ns, execPodName, fmt.Sprintf("http://%s:%d/debug/pprof/heap", host, statsPort), username, password) + results, err := getAuthenticatedURLViaPod(ns, execPodName, fmt.Sprintf("http://%s:%d/debug/pprof/heap?debug=1", host, statsPort), username, password) o.Expect(err).NotTo(o.HaveOccurred()) o.Expect(results).To(o.ContainSubstring("# runtime.MemStats")) })