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

fanng pushed a commit to branch branch-backend-it-maintenance-routing-explicit
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to 
refs/heads/branch-backend-it-maintenance-routing-explicit by this push:
     new 8174013432 test(ci): isolate reusable workflow input issue
8174013432 is described below

commit 81740134329b16f553fb081fcbc9c4932271c52b
Author: fanng <[email protected]>
AuthorDate: Mon Mar 23 17:35:54 2026 +0800

    test(ci): isolate reusable workflow input issue
---
 .../workflows/backend-integration-test-action.yml  | 70 +++++++++-------------
 .github/workflows/backend-integration-test.yml     |  6 +-
 2 files changed, 31 insertions(+), 45 deletions(-)

diff --git a/.github/workflows/backend-integration-test-action.yml 
b/.github/workflows/backend-integration-test-action.yml
index 15a531511c..b9b275e21a 100644
--- a/.github/workflows/backend-integration-test-action.yml
+++ b/.github/workflows/backend-integration-test-action.yml
@@ -25,11 +25,6 @@ on:
         description: 'space-separated changed catalogs-contrib module names, 
or __all__'
         type: string
         default: ''
-      maintenance_only:
-        required: false
-        description: 'run only maintenance integration tests'
-        type: string
-        default: 'false'
 
 jobs:
   start-runner:
@@ -78,44 +73,37 @@ jobs:
             -PskipDockerTests=false
           )
 
-          if [ "${{ inputs.maintenance_only }}" = "true" ]; then
-            gradle_args+=(
-              :maintenance:optimizer:test
-              :maintenance:jobs:test
-            )
-          else
-            ALL_CONTRIB_MODULES="$(find catalogs-contrib -mindepth 1 -maxdepth 
1 -type d -name 'catalog-*' -exec basename {} \; | sort)"
-            CHANGED_CONTRIB_MODULES="${{ 
inputs.changed-catalogs-contrib-modules }}"
+          ALL_CONTRIB_MODULES="$(find catalogs-contrib -mindepth 1 -maxdepth 1 
-type d -name 'catalog-*' -exec basename {} \; | sort)"
+          CHANGED_CONTRIB_MODULES="${{ inputs.changed-catalogs-contrib-modules 
}}"
 
-            gradle_args+=(
-              test
-              -x :web:web:test
-              -x :web:integration-test:test
-              -x :web-v2:web:test
-              -x :web-v2:integration-test:test
-              -x :clients:client-python:test
-              -x :flink-connector:flink:test
-              -x :spark-connector:spark-common:test
-              -x :spark-connector:spark-3.3:test
-              -x :spark-connector:spark-3.4:test
-              -x :spark-connector:spark-3.5:test
-              -x :spark-connector:spark-runtime-3.3:test
-              -x :spark-connector:spark-runtime-3.4:test
-              -x :spark-connector:spark-runtime-3.5:test
-              -x :trino-connector:integration-test:test
-              -x :trino-connector:trino-connector:test
-              -x :authorizations:authorization-chain:test
-              -x :authorizations:authorization-ranger:test
-              -x :clients:cli:test
-            )
+          gradle_args+=(
+            test
+            -x :web:web:test
+            -x :web:integration-test:test
+            -x :web-v2:web:test
+            -x :web-v2:integration-test:test
+            -x :clients:client-python:test
+            -x :flink-connector:flink:test
+            -x :spark-connector:spark-common:test
+            -x :spark-connector:spark-3.3:test
+            -x :spark-connector:spark-3.4:test
+            -x :spark-connector:spark-3.5:test
+            -x :spark-connector:spark-runtime-3.3:test
+            -x :spark-connector:spark-runtime-3.4:test
+            -x :spark-connector:spark-runtime-3.5:test
+            -x :trino-connector:integration-test:test
+            -x :trino-connector:trino-connector:test
+            -x :authorizations:authorization-chain:test
+            -x :authorizations:authorization-ranger:test
+            -x :clients:cli:test
+          )
 
-            if [ "$CHANGED_CONTRIB_MODULES" != "__all__" ]; then
-              for module in $ALL_CONTRIB_MODULES; do
-                if [[ " $CHANGED_CONTRIB_MODULES " != *" $module "* ]]; then
-                  gradle_args+=("-x" ":catalogs-contrib:$module:test")
-                fi
-              done
-            fi
+          if [ "$CHANGED_CONTRIB_MODULES" != "__all__" ]; then
+            for module in $ALL_CONTRIB_MODULES; do
+              if [[ " $CHANGED_CONTRIB_MODULES " != *" $module "* ]]; then
+                gradle_args+=("-x" ":catalogs-contrib:$module:test")
+              fi
+            done
           fi
 
           printf './gradlew'
diff --git a/.github/workflows/backend-integration-test.yml 
b/.github/workflows/backend-integration-test.yml
index e0fbe679d8..8d71410d20 100644
--- a/.github/workflows/backend-integration-test.yml
+++ b/.github/workflows/backend-integration-test.yml
@@ -126,13 +126,12 @@ jobs:
             backend: 'postgresql'
           - test-mode: 'deploy'
             backend: 'h2'
-    uses: 
apache/gravitino/.github/workflows/backend-integration-test-action.yml@branch-backend-it-maintenance-routing-explicit
+    uses: ./.github/workflows/backend-integration-test-action.yml
     with:
       architecture: ${{ matrix.architecture }}
       java-version: ${{ matrix.java-version }}
       backend: ${{ matrix.backend }}
       test-mode: ${{ matrix.test-mode }}
-      maintenance_only: ${{ 
needs.changes.outputs.maintenance_module_only_changes }}
       changed-catalogs-contrib-modules: ${{ 
needs.changes.outputs.changed_catalogs_contrib_modules }}
 
   BackendIT-on-pr:
@@ -151,11 +150,10 @@ jobs:
             backend: 'postgresql'
           - test-mode: 'deploy'
             backend: 'h2'
-    uses: 
apache/gravitino/.github/workflows/backend-integration-test-action.yml@branch-backend-it-maintenance-routing-explicit
+    uses: ./.github/workflows/backend-integration-test-action.yml
     with:
       architecture: ${{ matrix.architecture }}
       java-version: ${{ matrix.java-version }}
       backend: ${{ matrix.backend }}
       test-mode: ${{ matrix.test-mode }}
-      maintenance_only: ${{ 
needs.changes.outputs.maintenance_module_only_changes }}
       changed-catalogs-contrib-modules: ${{ 
needs.changes.outputs.changed_catalogs_contrib_modules }}

Reply via email to