Merge pull request #149 from Leaseweb/148-syncer-job-tolerations

Fix Syncer job tolerations + Adding Chart Lint - PR 148
Leaseweb · Jul 24, 2024 · 7e48373 · 7e48373
2 parents 3ba6b04 + 68124f2
commit 7e48373
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/.github/workflows/chart-lints.yaml b/.github/workflows/chart-lints.yaml
@@ -32,4 +32,7 @@ jobs:
           changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
           if [[ -n "$changed" ]]; then
             echo "changed=true" >> "$GITHUB_OUTPUT"
-          fi
+          fi
+      - name: Run chart-testing (lint)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: ct lint --chart-dirs=charts --target-branch ${{ github.event.repository.default_branch }}
diff --git a/charts/cloudstack-csi/Chart.yaml b/charts/cloudstack-csi/Chart.yaml
@@ -2,10 +2,14 @@ apiVersion: v2
 name: cloudstack-csi
 description: A Helm chart for CloudStack CSI driver
 type: application
-version: 2.0.1
+version: 2.0.2
 appVersion: 0.6.1
 sources:
   - https://github.com/Leaseweb/cloudstack-csi-driver
 keywords:
   - csi
   - cloudStack
+maintainers:
+  - name: hrak
+  - name: FarnazBGH
+  - name: lanord
diff --git a/charts/cloudstack-csi/templates/syncer-job.yaml b/charts/cloudstack-csi/templates/syncer-job.yaml
@@ -19,7 +19,7 @@ spec:
       securityContext: {{- toYaml .Values.syncer.podSecurityContext | nindent 8 }}
       serviceAccountName: {{ .Values.syncer.serviceAccount.name }}
       tolerations:
-        {{- with .Values.node.tolerations }}
+        {{- with .Values.syncer.tolerations }}
           {{- toYaml . | nindent 8 }}
           {{- end }}
       {{- with .Values.syncer.affinity }}