(gravitino) branch main updated: [#6971][followup] build: Fix incorrect target branch variable in chart CI (#6984)

[roryqi]

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

roryqi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 281c836e94 [#6971][followup] build: Fix incorrect target branch 
variable in chart CI (#6984)
281c836e94 is described below

commit 281c836e94869a649715e40920b56c406fc783c4
Author: Danhua Wang <dan...@datastrato.com>
AuthorDate: Thu Apr 17 15:00:24 2025 +0800

    [#6971][followup] build: Fix incorrect target branch variable in chart CI 
(#6984)
    
    ### What changes were proposed in this pull request?
    
    Fix incorrect target branch variable in chart CI
    
    ### Why are the changes needed?
    
    Fix: #6971
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    
    CI
---
 .github/workflows/chart-test.yaml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/chart-test.yaml 
b/.github/workflows/chart-test.yaml
index 38b09c8588..0679c72c19 100644
--- a/.github/workflows/chart-test.yaml
+++ b/.github/workflows/chart-test.yaml
@@ -55,7 +55,7 @@ jobs:
       - name: List changed
         id: list-changed
         run: |
-          changed=$(ct list-changed --chart-dirs=dev/charts --target-branch 
${{ github.event.repository.default_branch }})
+          changed=$(ct list-changed --chart-dirs=dev/charts --target-branch 
${{ github.event.pull_request.base.ref }})
           if [[ -n "$changed" ]]; then
             echo "changed=true" >> "$GITHUB_OUTPUT"
           fi
@@ -64,7 +64,7 @@ jobs:
         if: steps.list-changed.outputs.changed == 'true'
         run: |
           helm repo add bitnami https://charts.bitnami.com/bitnami
-          ct lint --chart-dirs=dev/charts 
--chart-yaml-schema=dev/ci/chart_schema.yaml --lint-conf=dev/ci/lintconf.yaml 
--target-branch ${{ github.event.repository.default_branch }} 
+          ct lint --chart-dirs=dev/charts 
--chart-yaml-schema=dev/ci/chart_schema.yaml --lint-conf=dev/ci/lintconf.yaml 
--target-branch ${{ github.event.pull_request.base.ref }} 
 
       - name: Set up kind cluster
         if: steps.list-changed.outputs.changed == 'true'
@@ -78,8 +78,8 @@ jobs:
 
       - name: Run chart-testing (install default)
         if: steps.list-changed.outputs.changed == 'true'
-        run: ct install --chart-dirs=dev/charts --target-branch ${{ 
github.event.repository.default_branch }}
+        run: ct install --chart-dirs=dev/charts --target-branch ${{ 
github.event.pull_request.base.ref }}
 
       - name: Run chart-testing (install mysql enable)
         if: steps.list-changed.outputs.changed == 'true'
-        run: ct install --chart-dirs=dev/charts --helm-extra-set-args 
"--values dev/charts/gravitino/resources/scenarios/ci-values.yaml" 
--target-branch ${{ github.event.repository.default_branch }}
\ No newline at end of file
+        run: ct install --chart-dirs=dev/charts --helm-extra-set-args 
"--values dev/charts/gravitino/resources/scenarios/ci-values.yaml" 
--target-branch ${{ github.event.pull_request.base.ref }}
\ No newline at end of file