Skip to content

Commit e34e4fb

Browse files
committed
fix: use helm-values-schema-json-action for schema drift check
1 parent b7dc287 commit e34e4fb

1 file changed

Lines changed: 7 additions & 11 deletions

File tree

.github/workflows/_helm.yaml

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -63,21 +63,17 @@ jobs:
6363
- name: Checkout
6464
uses: actions/checkout@v6
6565

66-
- name: Set up Helm
67-
uses: azure/setup-helm@v5
68-
69-
- name: Install helm-values-schema-json plugin
70-
run: helm plugin install https://github.com/losisin/helm-values-schema-json
71-
72-
- name: Generate schema
73-
run: helm schema -input charts/${{ matrix.chart }}/values.yaml -output /tmp/values.schema.json
66+
- name: Generate schema for ${{ matrix.chart }}
67+
uses: losisin/helm-values-schema-json-action@v2
68+
with:
69+
input: charts/${{ matrix.chart }}/values.yaml
70+
output: charts/${{ matrix.chart }}/values.schema.json
7471

7572
- name: Check for schema drift
7673
run: |
77-
if ! diff -q /tmp/values.schema.json charts/${{ matrix.chart }}/values.schema.json > /dev/null 2>&1; then
74+
if ! git diff --exit-code charts/${{ matrix.chart }}/values.schema.json; then
7875
echo ""
7976
echo "ERROR: values.schema.json for ${{ matrix.chart }} is out of date."
80-
echo "Run 'helm schema -input charts/${{ matrix.chart }}/values.yaml -output charts/${{ matrix.chart }}/values.schema.json' and commit."
81-
diff /tmp/values.schema.json charts/${{ matrix.chart }}/values.schema.json || true
77+
echo "Run the helm-values-schema-json action locally and commit the result."
8278
exit 1
8379
fi

0 commit comments

Comments
 (0)