[incubator-opendal] branch main updated: ci(bindings/java): skip RedisServiceTest on macos and windows (#3054)

[xuanwo]

This is an automated email from the ASF dual-hosted git repository.

xuanwo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-opendal.git


The following commit(s) were added to refs/heads/main by this push:
     new 9ac7bb9d8 ci(bindings/java): skip RedisServiceTest on macos and 
windows (#3054)
9ac7bb9d8 is described below

commit 9ac7bb9d85220a013ddda8d55a8403a8a3499c08
Author: tison <wander4...@gmail.com>
AuthorDate: Wed Sep 13 22:06:20 2023 +0800

    ci(bindings/java): skip RedisServiceTest on macos and windows (#3054)
---
 .github/workflows/bindings_java.yml                       | 15 ++++++++-------
 .github/workflows/publish.yml                             |  4 +---
 .../test/java/org/apache/opendal/RedisServiceTest.java    |  3 ++-
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/bindings_java.yml 
b/.github/workflows/bindings_java.yml
index 169b2935b..4d5803fcd 100644
--- a/.github/workflows/bindings_java.yml
+++ b/.github/workflows/bindings_java.yml
@@ -79,11 +79,12 @@ jobs:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
       - name: Build and test
         working-directory: bindings/java
-        # `install` is required to verify reproducible builds:
+        # `mvn install` and `mvn artifact:compare` are required to verify 
reproducible builds:
         # 
https://maven.apache.org/guides/mini/guide-reproducible-builds.html#how-to-test-my-maven-build-reproducibility
-        run: ./mvnw clean install
-      - name: Verify reproducibility
-        working-directory: bindings/java
-        # `artifact:compare` is required to verify reproducible builds:
-        # 
https://maven.apache.org/guides/mini/guide-reproducible-builds.html#how-to-test-my-maven-build-reproducibility
-        run: ./mvnw verify artifact:compare
+        shell: bash
+        run: |
+          if [[ "${{ runner.os }}" != Linux ]]; then
+            export NO_DOCKER=true
+          fi
+          ./mvnw clean install -DskipTests
+          ./mvnw verify artifact:compare
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
index b754e4e1d..a00ab9883 100644
--- a/.github/workflows/publish.yml
+++ b/.github/workflows/publish.yml
@@ -20,9 +20,7 @@ name: Publish
 on:
   push:
     tags:
-      - '*'
-    tags-ignore:
-      - '*rc*'
+      - 'v[0-9]+\.[0-9]+\.[0-9]+'
 
 jobs:
   publish:
diff --git 
a/bindings/java/src/test/java/org/apache/opendal/RedisServiceTest.java 
b/bindings/java/src/test/java/org/apache/opendal/RedisServiceTest.java
index f5751d931..b93a40115 100644
--- a/bindings/java/src/test/java/org/apache/opendal/RedisServiceTest.java
+++ b/bindings/java/src/test/java/org/apache/opendal/RedisServiceTest.java
@@ -26,13 +26,14 @@ import java.util.Map;
 import java.util.concurrent.CompletionException;
 import lombok.Cleanup;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.DisabledIfEnvironmentVariable;
 import org.testcontainers.containers.GenericContainer;
 import org.testcontainers.junit.jupiter.Container;
 import org.testcontainers.junit.jupiter.Testcontainers;
 
 @Testcontainers
+@DisabledIfEnvironmentVariable(named = "NO_DOCKER", matches = "true")
 public class RedisServiceTest {
-
     @Container
     private final GenericContainer<?> redisContainer = new 
GenericContainer<>("redis:7.2.1").withExposedPorts(6379);