Skip to content

Commit

Permalink
Correct the url for the memory stats in the router test
Browse files Browse the repository at this point in the history
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 #15356
  • Loading branch information
knobunc committed Jul 26, 2017
1 parent d71cd79 commit 9f6702a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/extended/router/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -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"))
})
Expand Down

0 comments on commit 9f6702a

Please sign in to comment.