Enforce value files must match schema

WyriHaximusNet · Nov 23, 2024 · 71cec5f · 71cec5f
1 parent c82af70
commit 71cec5f
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 24 deletions.
diff --git a/.github/workflows/helm-charts-test.yaml b/.github/workflows/helm-charts-test.yaml
@@ -18,33 +18,28 @@ jobs:
       - name: Lint charts
         id: lint
         run: ct lint --all
-#  collect-values-files-to-validate:
-#    runs-on: ubuntu-latest
-#    name: Validate values schema json
-#    steps:
-#      - name: Checkout
-#        uses: actions/checkout@v4
-#      - name: Validate values schema json
-#        uses: losisin/helm-values-schema-json-action@v1
-#        with:
-#          input: values.yaml
-#          fail-on-diff: true
-#  validate-values-schema:
-#    runs-on: ubuntu-latest
-#    name: Validate values schema json
-#    steps:
-#      - name: Checkout
-#        uses: actions/checkout@v4
-#      - name: Validate values schema json
-#        uses: losisin/helm-values-schema-json-action@v1
-#        with:
-#          input: values.yaml
-#          fail-on-diff: true
+  validate-values-schema:
+    runs-on: ubuntu-latest
+    name: Validate values schema json
+    strategy:
+      matrix:
+        include:
+          - chart: cron-jobs
+            values: ci/sleep-values.yaml
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Validate values schema json
+        uses: losisin/helm-values-schema-json-action@v1
+        with:
+          input: charts/${{ matrix.chart }}/${{ matrix.values }}
+          output: charts/${{ matrix.chart }}/values.schema.json
+          fail-on-diff: true
   test:
     runs-on: ubuntu-latest
     needs:
       - lint
-#      - validate-values-schema
+      - validate-values-schema
     strategy:
       fail-fast: false
       matrix:

diff --git a/charts/cron-jobs/ci/sleep-values.yaml b/charts/cron-jobs/ci/sleep-values.yaml
@@ -1,7 +1,7 @@
 # List of cronjobs to create
 jobs:
   # Cronjob's name
-  - name: sleep  # @schema required:true
+  - naam: sleep  # @schema required:true
     # Cronjob's schedule following [https://crontab.guru/](crontab syntax)
     schedule: "* * * * *"  # @schema required:true
     # Any labels to apply to the created cronjob