From 7c05607aca39d3e6c0ff9aa0f3c21726dadae408 Mon Sep 17 00:00:00 2001 From: Robert Bailey Date: Wed, 10 Feb 2016 14:08:36 -0800 Subject: [PATCH] Revert #19469 so re-enable the metrics grabbing test on GKE. --- test/e2e/metrics_grabber_test.go | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/test/e2e/metrics_grabber_test.go b/test/e2e/metrics_grabber_test.go index 888d7f754a3..6620771348f 100644 --- a/test/e2e/metrics_grabber_test.go +++ b/test/e2e/metrics_grabber_test.go @@ -100,15 +100,12 @@ var _ = Describe("MetricsGrabber", func() { }) It("should grab all metrics from a Kubelet.", func() { - // We run this test only on GCE, as for some reason it flakes in GKE #19468 - if providerIs("gce") { - By("Proxying to Node through the API server") - nodes := ListSchedulableNodesOrDie(c) - Expect(nodes.Items).NotTo(BeEmpty()) - response, err := grabber.GrabFromKubelet(nodes.Items[0].Name) - expectNoError(err) - checkNecessaryMetrics(metrics.Metrics(response), metrics.NecessaryKubeletMetrics) - } + By("Proxying to Node through the API server") + nodes := ListSchedulableNodesOrDie(c) + Expect(nodes.Items).NotTo(BeEmpty()) + response, err := grabber.GrabFromKubelet(nodes.Items[0].Name) + expectNoError(err) + checkNecessaryMetrics(metrics.Metrics(response), metrics.NecessaryKubeletMetrics) }) It("should grab all metrics from a Scheduler.", func() {