You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by sh...@apache.org on 2022/09/01 07:03:54 UTC
[bookkeeper] branch master updated: Skip runs the workflow when only have site related changes (#3462)
This is an automated email from the ASF dual-hosted git repository.
shoothzj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new 8954d9848e Skip runs the workflow when only have site related changes (#3462)
8954d9848e is described below
commit 8954d9848e63ec5d32d1ba4bde26db1df2241801
Author: Yong Zhang <zh...@gmail.com>
AuthorDate: Thu Sep 1 15:03:47 2022 +0800
Skip runs the workflow when only have site related changes (#3462)
---
*Motivation*
We have enabled the requirement of the workflow before merging a
PR. We should skip checking the sites related changes because they
are non-code change.
*Modification*
- Using apache/pulsar-test-infra/paths-filter@master to filter the files
---
.github/workflows/backward-compat-tests.yml | 16 ++++++++++++++--
.github/workflows/bookie-tests.yml | 15 +++++++++++++--
.github/workflows/client-tests.yml | 13 +++++++++++--
.github/workflows/compatibility-check-java11.yml | 13 +++++++++++--
.github/workflows/compatibility-check-java17.yml | 13 +++++++++++--
.github/workflows/compatibility-check-java8.yml | 13 +++++++++++--
.github/workflows/integration-tests.yml | 15 +++++++++++++--
.github/workflows/macos-build.yml | 13 +++++++++++--
.github/workflows/pr-validation.yml | 15 +++++++++++++--
.github/workflows/remaining-tests.yml | 13 +++++++++++--
.github/workflows/replication-tests.yml | 13 +++++++++++--
.github/workflows/stream-tests.yml | 14 ++++++++++++--
.github/workflows/tls-tests.yml | 13 +++++++++++--
.github/workflows/windows-build.yml | 14 ++++++++++++--
14 files changed, 165 insertions(+), 28 deletions(-)
diff --git a/.github/workflows/backward-compat-tests.yml b/.github/workflows/backward-compat-tests.yml
index cd2fab8862..2743bedb5f 100644
--- a/.github/workflows/backward-compat-tests.yml
+++ b/.github/workflows/backward-compat-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,15 +62,20 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 1.8
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v1
with:
java-version: 1.8
- name: Install Module
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu clean install -DskipTests
- name: Test current server with old clients
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -DintegrationTests -pl :backward-compat-current-server-old-clients test
- name: Test progressive upgrade
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -DintegrationTests -pl :upgrade test
- name: Other tests
+ if: steps.changes.outputs.site3 != 'true'
run: |
mvn -B -nsu -DintegrationTests -pl :bc-non-fips,:hierarchical-ledger-manager,:hostname-bookieid,:old-cookie-new-cluster,:recovery-no-password,:upgrade-direct,:yahoo-custom-version test
diff --git a/.github/workflows/bookie-tests.yml b/.github/workflows/bookie-tests.yml
index 18a45dba14..1adaff13c5 100644
--- a/.github/workflows/bookie-tests.yml
+++ b/.github/workflows/bookie-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,18 +62,22 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Maven build bookkeeper-server
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -am -pl bookkeeper-server clean install -DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Run bookie tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -pl bookkeeper-server test -Dtest="org.apache.bookkeeper.bookie.**" -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Run shell tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -pl tests/scripts install -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/client-tests.yml b/.github/workflows/client-tests.yml
index b5103fc04b..ee1aac4a29 100644
--- a/.github/workflows/client-tests.yml
+++ b/.github/workflows/client-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,11 +62,13 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Run client tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -am -nsu -pl bookkeeper-server clean install test -Dtest="org.apache.bookkeeper.client.**" -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/compatibility-check-java11.yml b/.github/workflows/compatibility-check-java11.yml
index 50860b42bd..ae184d48f5 100644
--- a/.github/workflows/compatibility-check-java11.yml
+++ b/.github/workflows/compatibility-check-java11.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,11 +62,13 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Build with Maven
+ if: steps.changes.outputs.site3 != 'true'
run: mvn clean package -B -nsu -DskipBookKeeperServerTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: print JVM thread dumps when cancelled
if: cancelled()
diff --git a/.github/workflows/compatibility-check-java17.yml b/.github/workflows/compatibility-check-java17.yml
index 55a0209843..bdea73ecde 100644
--- a/.github/workflows/compatibility-check-java17.yml
+++ b/.github/workflows/compatibility-check-java17.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
env:
MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 -Dmaven.wagon.http.retryHandler.count=3
@@ -43,7 +41,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -54,11 +61,13 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 17
- name: Build with Maven
+ if: steps.changes.outputs.site3 != 'true'
run: mvn clean package -B -nsu -DskipBookKeeperServerTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: print JVM thread dumps when cancelled
if: cancelled()
diff --git a/.github/workflows/compatibility-check-java8.yml b/.github/workflows/compatibility-check-java8.yml
index c7ed72afe6..d738cafcfa 100644
--- a/.github/workflows/compatibility-check-java8.yml
+++ b/.github/workflows/compatibility-check-java8.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,10 +62,12 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 1.8
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v1
with:
java-version: 1.8
- name: Build with Maven
+ if: steps.changes.outputs.site3 != 'true'
run: mvn clean package spotbugs:check -B -nsu -DskipBookKeeperServerTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: print JVM thread dumps when cancelled
diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml
index baae816354..90b925608f 100644
--- a/.github/workflows/integration-tests.yml
+++ b/.github/workflows/integration-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,18 +62,22 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Build with Maven
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu clean install -Pdocker -DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Run metadata driver tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -f metadata-drivers/pom.xml test -DintegrationTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Run all integration tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -f tests/pom.xml test -DintegrationTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: print JVM thread dumps when cancelled
diff --git a/.github/workflows/macos-build.yml b/.github/workflows/macos-build.yml
index 726e817ead..b1f47cec96 100644
--- a/.github/workflows/macos-build.yml
+++ b/.github/workflows/macos-build.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
jobs:
test:
@@ -40,7 +38,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -51,10 +58,12 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v3
with:
distribution: 'temurin'
java-version: 11
- name: mvn package
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu clean package -DskipTests
diff --git a/.github/workflows/pr-validation.yml b/.github/workflows/pr-validation.yml
index ee80389228..1dd002bd3e 100644
--- a/.github/workflows/pr-validation.yml
+++ b/.github/workflows/pr-validation.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,15 +62,19 @@ jobs:
key: ${{ runner.os }}-bookkeeper-all-${{ hashFiles('**/pom.xml') }}
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Validate pull request
+ if: steps.changes.outputs.site3 != 'true'
run: mvn clean -B -nsu apache-rat:check checkstyle:check spotbugs:check package -Ddistributedlog -DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Check license files
+ if: steps.changes.outputs.site3 != 'true'
run: dev/check-all-licenses
- name: Generate Javadoc
+ if: steps.changes.outputs.site3 != 'true'
run: mvn clean -B -nsu -am -pl bookkeeper-common,bookkeeper-server,:bookkeeper-stats-api,:bookkeeper-stats-providers,:codahale-metrics-provider,:prometheus-metrics-provider install javadoc:aggregate -DskipTests -Pdelombok
diff --git a/.github/workflows/remaining-tests.yml b/.github/workflows/remaining-tests.yml
index f778dda463..106428818f 100644
--- a/.github/workflows/remaining-tests.yml
+++ b/.github/workflows/remaining-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -57,11 +64,13 @@ jobs:
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Run remaining tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -am -pl bookkeeper-server clean install test -Dtest="!org.apache.bookkeeper.client.**,!org.apache.bookkeeper.bookie.**,!org.apache.bookkeeper.replication.**,!org.apache.bookkeeper.tls.**" -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/replication-tests.yml b/.github/workflows/replication-tests.yml
index 086769b06e..a6f8ead918 100644
--- a/.github/workflows/replication-tests.yml
+++ b/.github/workflows/replication-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -56,11 +63,13 @@ jobs:
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Run replication tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -am -pl bookkeeper-server clean install test -Dtest="org.apache.bookkeeper.replication.**" -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/stream-tests.yml b/.github/workflows/stream-tests.yml
index 3d9c22f2e3..19a6bda6e8 100644
--- a/.github/workflows/stream-tests.yml
+++ b/.github/workflows/stream-tests.yml
@@ -24,8 +24,6 @@ on:
pull_request:
branches:
- master
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -43,7 +41,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,13 +62,16 @@ jobs:
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Build with Maven
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu clean install -DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Run StreamStorage tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu -f stream/pom.xml verify -DstreamTests -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/tls-tests.yml b/.github/workflows/tls-tests.yml
index 79aacfc9c5..fa4c56010d 100644
--- a/.github/workflows/tls-tests.yml
+++ b/.github/workflows/tls-tests.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
workflow_dispatch:
env:
@@ -44,7 +42,16 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -56,11 +63,13 @@ jobs:
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v2
with:
distribution: 'temurin'
java-version: 11
- name: Run tls tests
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -am -nsu -pl bookkeeper-server clean install test -Dtest="org.apache.bookkeeper.tls.**" -DfailIfNoTests=false -Dorg.slf4j.simpleLogger.defaultLogLevel=INFO
- name: Aggregates all test reports to ./test-reports and ./surefire-reports directories If failure
diff --git a/.github/workflows/windows-build.yml b/.github/workflows/windows-build.yml
index e75c656f24..c3510b0b9e 100644
--- a/.github/workflows/windows-build.yml
+++ b/.github/workflows/windows-build.yml
@@ -25,8 +25,6 @@ on:
branches:
- master
- branch-*
- paths-ignore:
- - 'site3/**'
jobs:
test:
@@ -40,10 +38,20 @@ jobs:
- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
+ - name: Detect changed files
+ id: changes
+ uses: apache/pulsar-test-infra/paths-filter@master
+ with:
+ filters: |
+ site3:
+ - 'site3/**'
+
- name: Install mingw
+ if: steps.changes.outputs.site3 != 'true'
run: choco install mingw
- name: Cache local Maven repository
+ if: steps.changes.outputs.site3 != 'true'
id: cache
uses: actions/cache@v2
with:
@@ -55,10 +63,12 @@ jobs:
- name: Set up JDK 11
+ if: steps.changes.outputs.site3 != 'true'
uses: actions/setup-java@v3
with:
distribution: 'temurin'
java-version: 11
- name: mvn package
+ if: steps.changes.outputs.site3 != 'true'
run: mvn -B -nsu clean package -DskipTests