Enable isolate mode for github actions

ydb-platform · Apr 27, 2024 · 575e54c · 575e54c
1 parent 3ef1860
commit 575e54c
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -19,7 +19,7 @@ jobs:
         java: [ '8', '11', '17']
 
     env:
-        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always
+        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always -DYDB_DOCKER_ISOLATION=true
 
     steps:
       - uses: actions/checkout@v4
@@ -41,7 +41,7 @@ jobs:
     needs: build
 
     env:
-        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always
+        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always -DYDB_DOCKER_ISOLATION=true
 
     steps:
       - uses: actions/checkout@v4

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -19,7 +19,7 @@ jobs:
         java: [ '8', '11', '17']
 
     env:
-        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always
+        MAVEN_ARGS: --batch-mode --update-snapshots -Dstyle.color=always -DYDB_DOCKER_ISOLATION=true
 
     steps:
       - uses: actions/checkout@v4

diff --git a/coordination/pom.xml b/coordination/pom.xml
@@ -47,6 +47,12 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <environmentVariables>
+                        <YDB_DOCKER_IMAGE>cr.yandex/yc/yandex-docker-local-ydb:trunk</YDB_DOCKER_IMAGE>
+                        <TESTCONTAINERS_REUSE_ENABLE>true</TESTCONTAINERS_REUSE_ENABLE>
+                    </environmentVariables>
+                </configuration>
             </plugin>
         </plugins>
     </build>

diff --git a/table/pom.xml b/table/pom.xml
@@ -50,7 +50,6 @@
                 <configuration>
                     <environmentVariables>
                         <YDB_DOCKER_IMAGE>cr.yandex/yc/yandex-docker-local-ydb:trunk</YDB_DOCKER_IMAGE>
-                        <YDB_DOCKER_ISOLATION>true</YDB_DOCKER_ISOLATION>
                         <TESTCONTAINERS_REUSE_ENABLE>true</TESTCONTAINERS_REUSE_ENABLE>
                     </environmentVariables>
                 </configuration>

diff --git a/tests/common/src/main/java/tech/ydb/test/integration/docker/GrpcProxyServer.java b/tests/common/src/main/java/tech/ydb/test/integration/docker/GrpcProxyServer.java
@@ -183,8 +183,8 @@ private class ProxyRegistry extends HandlerRegistry {
 
         @Override
         public ServerMethodDefinition<?, ?> lookupMethod(String methodName, String authority) {
-            logger.info("lookup method {}", methodName);
             if (DiscoveryServiceGrpc.getListEndpointsMethod().getFullMethodName().equals(methodName)) {
+                logger.info("use custom proxy for method {}", methodName);
                 return new DiscoveryServiceProxy(endpoint).toMethodDefinition();
             }