From 77e3202f0b70ab32af2e8ebf4b4937e49f9bd3bf Mon Sep 17 00:00:00 2001
From: Ran Nozik <ran@gethelios.dev>
Date: Sun, 11 Sep 2022 19:13:32 +0300
Subject: [PATCH] disable metrics tests

---
 .../smoketest/PrometheusSmokeTest.groovy             |  3 +++
 .../smoketest/SpringBootSmokeTest.groovy             | 12 ++++++------
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/PrometheusSmokeTest.groovy b/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/PrometheusSmokeTest.groovy
index ce86e16160db..7b3b375bdccb 100644
--- a/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/PrometheusSmokeTest.groovy
+++ b/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/PrometheusSmokeTest.groovy
@@ -6,6 +6,8 @@
 package io.opentelemetry.smoketest
 
 import io.opentelemetry.testing.internal.armeria.client.WebClient
+import spock.lang.Ignore
+
 import java.time.Duration
 import spock.lang.IgnoreIf
 
@@ -33,6 +35,7 @@ class PrometheusSmokeTest extends SmokeTest {
     return [PROMETHEUS_PORT]
   }
 
+  @Ignore // We've disabled the metrics provider
   def "Should export metrics"(int jdk) {
     setup:
     startTarget(jdk)
diff --git a/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/SpringBootSmokeTest.groovy b/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/SpringBootSmokeTest.groovy
index b013a87a0a73..b03dc85dd709 100644
--- a/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/SpringBootSmokeTest.groovy
+++ b/smoke-tests/src/test/groovy/io/opentelemetry/smoketest/SpringBootSmokeTest.groovy
@@ -75,12 +75,12 @@ class SpringBootSmokeTest extends SmokeTest {
       .collect(toSet())
     loggedTraceIds == spanTraceIds
 
-    then: "JVM metrics are exported"
-    def metrics = new MetricsInspector(waitForMetrics())
-    metrics.hasMetricsNamed("process.runtime.jvm.memory.init")
-    metrics.hasMetricsNamed("process.runtime.jvm.memory.usage")
-    metrics.hasMetricsNamed("process.runtime.jvm.memory.committed")
-    metrics.hasMetricsNamed("process.runtime.jvm.memory.limit")
+//    then: "JVM metrics are exported"
+//    def metrics = new MetricsInspector(waitForMetrics())
+//    metrics.hasMetricsNamed("process.runtime.jvm.memory.init")
+//    metrics.hasMetricsNamed("process.runtime.jvm.memory.usage")
+//    metrics.hasMetricsNamed("process.runtime.jvm.memory.committed")
+//    metrics.hasMetricsNamed("process.runtime.jvm.memory.limit")
 
     cleanup:
     stopTarget()