Check OS for scraping host

quarkusio · Feb 3, 2025 · e8635a1 · e8635a1
1 parent 4d42956
commit e8635a1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/extensions/observability-devservices/testcontainers/pom.xml b/extensions/observability-devservices/testcontainers/pom.xml
@@ -13,6 +13,10 @@
     <name>Quarkus - Observability Dev Services - Testcontainers</name>
 
     <dependencies>
+        <dependency>
+            <groupId>io.quarkus</groupId>
+            <artifactId>quarkus-devtools-utilities</artifactId>
+        </dependency>
         <dependency>
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-devservices-common</artifactId>

diff --git a/...s/testcontainers/src/main/java/io/quarkus/observability/testcontainers/LgtmContainer.java b/...s/testcontainers/src/main/java/io/quarkus/observability/testcontainers/LgtmContainer.java
@@ -16,6 +16,7 @@
 import io.quarkus.observability.common.config.AbstractGrafanaConfig;
 import io.quarkus.observability.common.config.LgtmConfig;
 import io.quarkus.runtime.LaunchMode;
+import io.quarkus.utilities.OS;
 
 public class LgtmContainer extends GrafanaContainer<LgtmContainer, LgtmConfig> {
     protected static final String LGTM_NETWORK_ALIAS = "ltgm.testcontainer.docker";
@@ -177,8 +178,11 @@ private String getPrometheusConfig() {
             Optional<Integer> optionalValue = runtimeConfig.getOptionalValue(httpPortKey, Integer.class);
             int httpPort = optionalValue.orElse(isTest ? 8081 : 8080); // when not set use default
 
+            // try 'localhost' on Linux / CI ...
+            String host = OS.determineOS() == OS.LINUX ? "localhost" : "host.docker.internal";
+
             prometheusConfig += String.format(PROMETHEUS_CONFIG_SCRAPE, config.serviceName(), rootPath, metricsPath, scraping,
-                    "host.docker.internal", httpPort);
+                    host, httpPort);
         }
         return prometheusConfig;
     }