Bump sandbox acquisition timeout of trigger service tests to 1m (#11764)

This matches the timeouts we set in SandboxNextFixture and I’ve seen the 30s timeouts be insufficient on CI. changelog_begin changelog_end
digital-asset · Nov 18, 2021 · 869b805 · 869b805
1 parent 4106222
commit 869b805
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/...ervice/src/test/scala/com/digitalasset/daml/lf/engine/trigger/TriggerServiceFixture.scala b/...ervice/src/test/scala/com/digitalasset/daml/lf/engine/trigger/TriggerServiceFixture.scala
@@ -306,16 +306,20 @@ trait SandboxFixture extends BeforeAndAfterAll with AbstractAuthFixture with Akk
     implicit val context: ResourceContext = ResourceContext(system.dispatcher)
     // The owner spins up its own actor system which avoids deadlocks
     // during shutdown.
-    resource = new OwnedResource(for {
-      // We must provide a random database if none is provided.
-      // The default is to always use the same index database URL, which means that tests can
-      // share an index. As you can imagine, this causes all manner of issues, the most important
-      // of which is that the ledger and index databases will be out of sync.
-      jdbcUrl <- SandboxBackend.H2Database.owner
-        .map(info => Some(info.jdbcUrl))
-      port <- new SandboxRunner(sandboxConfig.copy(jdbcUrl = jdbcUrl))
-      channel <- GrpcClientResource.owner(port)
-    } yield (port, channel))
+    resource = new OwnedResource(
+      for {
+        // We must provide a random database if none is provided.
+        // The default is to always use the same index database URL, which means that tests can
+        // share an index. As you can imagine, this causes all manner of issues, the most important
+        // of which is that the ledger and index databases will be out of sync.
+        jdbcUrl <- SandboxBackend.H2Database.owner
+          .map(info => Some(info.jdbcUrl))
+        port <- new SandboxRunner(sandboxConfig.copy(jdbcUrl = jdbcUrl))
+        channel <- GrpcClientResource.owner(port)
+      } yield (port, channel),
+      acquisitionTimeout = 1.minute,
+      releaseTimeout = 1.minute,
+    )
     resource.setup()
   }