Adjust timeouts for LGTM container and tests

Fixes quarkusio#45774 (maybe...)
gsmet · Jan 23, 2025 · b3cdc89 · b3cdc89
1 parent 202c65c
commit b3cdc89
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/.github/workflows/ci-actions-incremental.yml b/.github/workflows/ci-actions-incremental.yml
@@ -53,7 +53,7 @@ env:
   COMMON_MAVEN_ARGS: "-e -B --settings .github/mvn-settings.xml --fail-at-end"
   COMMON_TEST_MAVEN_ARGS: "-Dformat.skip -Denforcer.skip -DskipDocs -Dforbiddenapis.skip -DskipExtensionValidation -DskipCodestartValidation"
   NATIVE_TEST_MAVEN_ARGS: "-Dtest-containers -Dstart-containers -Dquarkus.native.native-image-xmx=6g -Dnative -Dnative.surefire.skip -Dno-descriptor-tests clean install -DskipDocs"
-  JVM_TEST_MAVEN_ARGS: "-Dtest-containers -Dstart-containers -Dquarkus.test.hang-detection-timeout=60"
+  JVM_TEST_MAVEN_ARGS: "-Dtest-containers -Dstart-containers -Dquarkus.test.hang-detection-timeout=300"
   PTS_MAVEN_ARGS: "-Ddevelocity.pts.enabled=${{ github.event_name == 'pull_request' && github.base_ref == 'main' && 'true' || 'false' }}"
   DB_USER: hibernate_orm_test
   DB_PASSWORD: hibernate_orm_test

diff --git a/...es/common/src/main/java/io/quarkus/observability/common/config/AbstractGrafanaConfig.java b/...es/common/src/main/java/io/quarkus/observability/common/config/AbstractGrafanaConfig.java
@@ -46,6 +46,6 @@ public int grafanaPort() {
 
     @Override
     public Duration timeout() {
-        return Duration.ofMinutes(1);
+        return Duration.ofMinutes(3);
     }
 }
diff --git a/...evservices/common/src/main/java/io/quarkus/observability/common/config/GrafanaConfig.java b/...evservices/common/src/main/java/io/quarkus/observability/common/config/GrafanaConfig.java
@@ -29,6 +29,6 @@ public interface GrafanaConfig extends ContainerConfig {
     /**
      * The timeout.
      */
-    @WithDefault("PT1M")
+    @WithDefault("PT3M")
     Duration timeout();
 }
diff --git a/...-tests/observability-lgtm/src/test/java/io/quarkus/observability/test/LgtmReloadTest.java b/...-tests/observability-lgtm/src/test/java/io/quarkus/observability/test/LgtmReloadTest.java
@@ -35,7 +35,7 @@ public void testReload() {
         poke("/reload");
         test.modifySourceFile(ReloadEndpoint.class, s -> s.replace("/reload", "/new"));
         poke("/new");
-        test.modifyResourceFile("application.properties", s -> s.replace("timeout=PT1M", "timeout=PT2M"));
+        test.modifyResourceFile("application.properties", s -> s.replace("timeout=PT3M", "timeout=PT4M"));
         poke("/new");
     }
 }
diff --git a/integration-tests/observability-lgtm/src/test/resources/application.properties b/integration-tests/observability-lgtm/src/test/resources/application.properties
@@ -9,4 +9,4 @@ quarkus.micrometer.export.otlp.enabled=true
 quarkus.micrometer.export.otlp.publish=true
 quarkus.micrometer.export.otlp.step=PT5S
 
-quarkus.observability.lgtm.timeout=PT1M
+quarkus.observability.lgtm.timeout=PT3M