Skip to content

Commit 472f63a

Browse files
committed
make sure helm dependencies get updated
1 parent dc94cec commit 472f63a

File tree

1 file changed

+26
-3
lines changed

1 file changed

+26
-3
lines changed

.github/workflows/mlflow-ci.yml

Lines changed: 26 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
1-
name: MLflow CI (Direct Commands)
1+
name: MLflow CI
22

33
on:
44
pull_request:
55
paths:
66
- 'applications/mlflow/charts/**'
77
- 'applications/mlflow/kots/**'
88
- 'applications/mlflow/tests/**'
9-
- '.github/workflows/mlflow-ci-direct.yml'
9+
- '.github/workflows/mlflow-ci.yml'
1010
push:
1111
branches:
1212
- main
1313
paths:
1414
- 'applications/mlflow/charts/**'
1515
- 'applications/mlflow/kots/**'
1616
- 'applications/mlflow/tests/**'
17-
- '.github/workflows/mlflow-ci-direct.yml'
17+
- '.github/workflows/mlflow-ci.yml'
1818

1919
env:
2020
APP_SLUG: diamon-mlflow
@@ -44,6 +44,13 @@ jobs:
4444
helm repo add minio https://operator.min.io/
4545
helm repo update
4646
47+
- name: Update Helm dependencies
48+
run: |
49+
echo "Updating Helm dependencies for all charts..."
50+
helm dependency update applications/mlflow/charts/infra
51+
helm dependency update applications/mlflow/charts/mlflow
52+
echo "Helm dependencies updated successfully."
53+
4754
- name: Lint charts
4855
run: |
4956
echo "Linting Helm charts..."
@@ -99,6 +106,13 @@ jobs:
99106
helm repo add minio https://operator.min.io/
100107
helm repo update
101108
109+
- name: Update Helm dependencies
110+
run: |
111+
echo "Updating Helm dependencies for all charts..."
112+
helm dependency update applications/mlflow/charts/infra
113+
helm dependency update applications/mlflow/charts/mlflow
114+
echo "Helm dependencies updated successfully."
115+
102116
- name: Package infra chart
103117
run: |
104118
helm package applications/mlflow/charts/infra -d applications/mlflow/kots/ -u
@@ -240,6 +254,15 @@ jobs:
240254
helm repo add minio https://operator.min.io/
241255
helm repo update
242256
257+
- name: Update Helm dependencies
258+
run: |
259+
# This step ensures dependencies are updated for local chart references
260+
# Note: For OCI charts, dependencies are already included in the packaged chart
261+
echo "Updating Helm dependencies for all charts..."
262+
helm dependency update applications/mlflow/charts/infra
263+
helm dependency update applications/mlflow/charts/mlflow
264+
echo "Helm dependencies updated successfully."
265+
243266
# Authenticate with the Replicated registry
244267
- name: Login to Replicated registry
245268
run: |

0 commit comments

Comments
 (0)