From 47fbf81219215a59fa5eb7e298ff5a8215dad419 Mon Sep 17 00:00:00 2001 From: Sergiusz Urbaniak Date: Mon, 15 Apr 2019 13:37:11 +0200 Subject: [PATCH] test/extended/prometheus: temporarily disable scheduler metrics checks --- test/extended/prometheus/prometheus.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/test/extended/prometheus/prometheus.go b/test/extended/prometheus/prometheus.go index 4ade3fff5a62..b95eeedf8846 100644 --- a/test/extended/prometheus/prometheus.go +++ b/test/extended/prometheus/prometheus.go @@ -128,9 +128,12 @@ var _ = g.Describe("[Feature:Prometheus][Conformance] Prometheus", func() { targets.Expect(labels{"job": "apiserver"}, "up", "^https://.*/metrics$"), // TODO: add openshift api // TODO: this should be https - // targets.Expect(labels{"job": "kube-controller-manager"}, "up", "^http://.*/metrics$"), - // TODO: check only for https after merging https://github.com/openshift/cluster-monitoring-operator/pull/308 - targets.Expect(labels{"job": "scheduler"}, "up", "^(http|https)://.*/metrics$"), + // TODO: check control plane once the following PRs land: + // - https://github.com/openshift/cluster-kube-scheduler-operator/pull/97 + // - https://github.com/openshift/installer/pull/1576 + // - https://github.com/openshift/cluster-monitoring-operator/pull/316 + // targets.Expect(labels{"job": "kube-controller-manager"}, "up", "^https://.*/metrics$"), + // targets.Expect(labels{"job": "scheduler"}, "up", "^https://.*/metrics$"), targets.Expect(labels{"job": "kube-state-metrics"}, "up", "^https://.*/metrics$"), // TODO: should probably be https targets.Expect(labels{"job": "cluster-version-operator"}, "up", "^http://.*/metrics$"),