Improve helm checking

Signed-off-by: Gavin Halliday <[email protected]>

Author: ghalliday

.github/workflows/test-helm.yml

@@ -38,7 +38,7 @@ jobs:
     name: "Check helm chart lint"
     runs-on: ubuntu-20.04
     needs: pre_job
-    if: ${{ github.repository == 'hpcc-systems/HPCC-Platform' && needs.pre_job.outputs.helm }}
+    #if: ${{ github.repository == 'hpcc-systems/HPCC-Platform' && needs.pre_job.outputs.helm }}
     steps:
       - uses: actions/checkout@v4
         with:
@@ -57,8 +57,50 @@ jobs:
       - name: Check for changes in helm output for default values
         working-directory: .
         run: |
-          helm template helm/hpcc > ${{ runner.temp }}/newoutput.txt
-          git fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin ${{ github.base_ref }}
-          git checkout ${{ github.base_ref }}
-          helm template helm/hpcc > ${{ runner.temp }}/oldoutput.txt
-          diff ${{ runner.temp }}/oldoutput.txt ${{ runner.temp }}/newoutput.txt
+          resultcode=0
+          mkdir ${{ runner.temp }}/new
+          mkdir ${{ runner.temp }}/old
+
+          #Generate the output for the default values file, and each of the specialised test files.
+          #Check to see if anything has changed between old and new, and report the differences
+          helm template helm/hpcc > ${{ runner.temp }}/new/output.txt
+          for file in testing/helm/tests/*.yaml
+          do
+            tail=$(basename $file)
+            helm template helm/hpcc --values $file > ${{ runner.temp }}/new/$tail.txt
+          done
+
+          git fetch --no-tags --prune --progress --no-recurse-submodules --quiet --depth=1 origin ${{ github.base_ref }}
+          echo git checkout ${{ github.base_ref }}^
+          git checkout ${{ github.base_ref }}^
+
+          helm template helm/hpcc > ${{ runner.temp }}/old/output.txt
+          for file in testing/helm/tests/*.yaml
+          do
+            tail=$(basename $file)
+            helm template helm/hpcc --values $file > ${{ runner.temp }}/old/$tail.txt
+          done
+
+          diff ${{ runner.temp }}/old/output.txt ${{ runner.temp }}/new/output.txt
+          if [ $? -ne 0 ]
+          then
+            resultcode=1
+          else
+            #Only check for differences in the specialised test files if the default values file is the same
+            for file in testing/helm/tests/*.yaml
+            do
+              tail=$(basename $file)
+              echo Check $tail
+              diff ${{ runner.temp }}/old/$tail.txt ${{ runner.temp }}/new/$tail.txt
+              if [ $? -ne 0 ]
+              then
+                resultcode=1
+              fi
+              echo old
+              grep "my-dali-plane" ${{ runner.temp }}/old/$tail.txt | cat
+              echo new
+              grep "my-dali-plane" ${{ runner.temp }}/new/$tail.txt | cat
+            done
+          fi
+
+          exit $resultcode

testing/helm/tests/implicitplanes.yaml

@@ -17,7 +17,7 @@ storage:
     storageClass: ""
     forcePermissions: true
     category: sasha
-  - name: my-dali-plane
+  - name: my-dali-planexx
     prefix: "/var/lib/HPCCSystems/dalistorage"
     storageSize: 1Gi
     storageClass: ""