Skip to content

Commit 4f1d8b6

Browse files
authored
Merge pull request #28 from emsearcy/helm-frozen-deps
Ensure Helm dependencies are correctly frozen
2 parents a38b650 + 7ebcbdb commit 4f1d8b6

File tree

5 files changed

+21
-9
lines changed

5 files changed

+21
-9
lines changed

.github/workflows/mega-linter.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,20 @@ jobs:
2929
with:
3030
fetch-depth: 0
3131

32+
# Installs packaged subcharts under charts/*/charts/*.tgz. Also
33+
# fails if the Chart.lock digest is wrong.
34+
- name: Helm dependency fetch
35+
shell: bash
36+
run: |
37+
set -euo pipefail
38+
for chart in charts/*; do
39+
if [ -f "${chart}/Chart.lock" ]; then
40+
yq -r '.dependencies[] | select(.repository | contains("oci://") == false) | "\(.name) \(.repository)"' \
41+
"${chart}/Chart.yaml" | xargs -L1 helm repo add --force-update
42+
helm dependency build "${chart}"
43+
fi
44+
done
45+
3246
# MegaLinter
3347
- name: MegaLinter
3448
id: ml

.github/workflows/release.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ jobs:
4343
4444
- name: Publish Chart to GHCR
4545
id: publish-ghcr
46-
uses: linuxfoundation/lfx-public-workflows/.github/actions/helm-chart-oci-publisher@a5d0271f539e3f69194cca94d9884914c3be088b
46+
uses: linuxfoundation/lfx-public-workflows/.github/actions/helm-chart-oci-publisher@e619121ece4ca4b1d6c89ade032f26105505756d
4747
with:
4848
name: ${{ steps.prepare.outputs.chart_name }}
4949
repository: ${{ github.repository }}/chart

.mega-linter.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,3 @@ SPELL_VALE_PRE_COMMANDS:
3737
FILTER_REGEX_EXCLUDE: '(templates/.*\.yml|templates/.*\.yaml)'
3838
KUBERNETES_DIRECTORY: charts/lfx-platform
3939
KUBERNETES_HELM_ARGUMENTS: charts/lfx-platform
40-
KUBERNETES_HELM_PRE_COMMANDS:
41-
- command: helm dependency update charts/lfx-platform

charts/lfx-platform/Chart.lock

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
dependencies:
22
- name: traefik
3-
repository: https://traefik.github.io/charts
3+
repository: oci://ghcr.io/traefik/helm
44
version: 36.2.0
55
- name: openfga
66
repository: https://openfga.github.io/helm-charts
77
version: 0.2.39
88
- name: heimdall
9-
repository: https://dadrus.github.io/heimdall/charts
9+
repository: oci://ghcr.io/dadrus/heimdall/chart
1010
version: 0.15.8
1111
- name: nats
1212
repository: https://nats-io.github.io/k8s/helm/charts/
@@ -32,5 +32,5 @@ dependencies:
3232
- name: trust-manager
3333
repository: https://charts.jetstack.io
3434
version: v0.18.0
35-
digest: sha256:591a323ff20b90c4b50fd92c63788e91d7e08ff2606f155af26e38527680a84d
36-
generated: "2025-08-06T11:35:55.1914-07:00"
35+
digest: sha256:62f9779ba2521042d18193fcaa7010ed905045c61579997d6551b2e9c23437fc
36+
generated: "2025-08-06T13:14:49.133573-07:00"

charts/lfx-platform/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,15 @@ version: 0.1.11
99
icon: https://github.com/linuxfoundation/lfx-v2-helm/raw/main/img/lfx-logo-color.svg
1010
dependencies:
1111
- name: traefik
12-
repository: https://traefik.github.io/charts
12+
repository: oci://ghcr.io/traefik/helm
1313
version: ~36.2.0
1414
condition: traefik.enabled
1515
- name: openfga
1616
repository: https://openfga.github.io/helm-charts
1717
version: ~0.2.37
1818
condition: openfga.enabled
1919
- name: heimdall
20-
repository: https://dadrus.github.io/heimdall/charts
20+
repository: oci://ghcr.io/dadrus/heimdall/chart
2121
version: ~0.15.6
2222
condition: heimdall.enabled
2323
- name: nats

0 commit comments

Comments
 (0)