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

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


The following commit(s) were added to refs/heads/main by this push:
     new dcd7cc63e CI: Make helm-CI run for all changes (#1611)
dcd7cc63e is described below

commit dcd7cc63e62e316c226e18a2c9a78ad097a98f83
Author: Robert Stupp <[email protected]>
AuthorDate: Wed May 21 09:57:15 2025 +0200

    CI: Make helm-CI run for all changes (#1611)
    
    Whether helm-charts work (or not) doesn't only depend on what's on the 
`helm/` directory, but also the production code. It's hard to figure out which 
changes could break the helm-charts or what the helm-charts rely on. Therefore 
it's likely better to run Helm-CI for all changes.
---
 .github/workflows/helm.yml | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/.github/workflows/helm.yml b/.github/workflows/helm.yml
index f56cc3447..899cfc039 100644
--- a/.github/workflows/helm.yml
+++ b/.github/workflows/helm.yml
@@ -57,16 +57,7 @@ jobs:
       - name: Set up chart-testing
         uses: helm/[email protected]
 
-      - name: Run chart-testing (list-changed)
-        id: list-changed
-        run: |
-          changed=$(ct list-changed --target-branch ${{ 
github.event.repository.default_branch }} --chart-dirs helm)
-          if [[ -n "$changed" ]]; then
-            echo "changed=true" >> "$GITHUB_OUTPUT"
-          fi
-
       - name: Run 'helm template' validation
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           cd helm/polaris
           for f in values.yaml ci/*.yaml; do
@@ -76,27 +67,22 @@ jobs:
           done
 
       - name: Run Helm unit tests
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           helm plugin install 
https://github.com/helm-unittest/helm-unittest.git || true
           helm unittest helm/polaris
 
       - name: Run chart-testing (lint)
-        if: steps.list-changed.outputs.changed == 'true'
         run: ct lint --target-branch ${{ 
github.event.repository.default_branch }} --debug --charts ./helm/polaris
 
       - name: Set up Minikube
-        if: steps.list-changed.outputs.changed == 'true'
         uses: medyagh/[email protected]
 
       - name: Print Docker info
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           docker -v
           minikube docker-env
 
       - name: Image build
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           eval $(minikube -p minikube docker-env)
           ./gradlew \
@@ -108,13 +94,11 @@ jobs:
           minikube image ls
 
       - name: Install fixtures
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           kubectl create namespace polaris-ns
           kubectl apply --namespace polaris-ns -f helm/polaris/ci/fixtures 
 
       - name: Run chart-testing (install)
-        if: steps.list-changed.outputs.changed == 'true'
         run: |
           ct install --target-branch ${{ 
github.event.repository.default_branch }} \
             --namespace polaris-ns \

Reply via email to