Merge pull request #32 from lanzhiwang/fix-chart-dependencies

update chart dependencies version in the script

Author: lanzhiwang

Makefile

@@ -0,0 +1,4 @@
+TOPDIR=$(shell dirname $(realpath $(firstword $(MAKEFILE_LIST))))
+
+update_chart_dependencies_version:
+	$(TOPDIR)/script/update_chart_dependencies_version.sh

charts/kubeflow/Chart.yaml

@@ -9,7 +9,6 @@ description: |
   but to provide a straightforward way to deploy best-of-breed
   open-source systems for ML to diverse infrastructures.
   Anywhere you are running Kubernetes, you should be able to run Kubeflow.
-  
 # 卸载须知：
 # 
 # 1. 执行 helm delete 之后，注意需要执行下面命令确保 profile 资源删除：
@@ -34,18 +33,18 @@ dependencies:
     version: "1.5.0"
     alias: cert-manager
     condition: cert-manager.enabled
+  - name: dex
+    version: "2.31.2"
+    condition: dex.enabled
   - name: istio
     version: "1.14.1"
     condition: istio.enabled
+  - name: knative-serving
+    version: 1.2.5
   - name: minio
     version: "1.0"
     condition: minio.enabled
-  - name: dex
-    version: "1.0"
-    condition: dex.enabled
-  - name: knative-serving
-    version: v0.22.1
 maintainers:
   - name: typhoonzero
     email: [email protected]
-appVersion: "1.5.0"
+appVersion: "1.6.1"

charts/kubeflow/charts/knative-serving/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v2
 name: knative-serving
-version: "v1.2.5"
+version: "1.2.5"
 kubeVersion: ">=1.17.0"
 description: "knative serving for kubeflow-chart."
 type: application
@@ -9,4 +9,4 @@ sources:
 maintainers:
   - name: typhoonzero
     email: [email protected]
-appVersion: "v1.2.5"
+appVersion: "1.2.5"

script/update_chart_dependencies_version.sh

@@ -0,0 +1,14 @@
+#!/usr/bin/env bash
+set -e
+
+root_dir=$(dirname $(dirname $(realpath $0)))
+chart_dir=${root_dir}/charts
+
+length=$(yq '.dependencies | length' ${chart_dir}/kubeflow/Chart.yaml)
+number=0
+while [[ ${number} -lt ${length} ]] ; do
+    name=$(echo ${number} | xargs -I {} yq e '.dependencies[{}].name' ${chart_dir}/kubeflow/Chart.yaml)
+    version=$(echo ${name} | xargs -I {} yq e '.version' ${chart_dir}/kubeflow/charts/{}/Chart.yaml)
+    eval "yq e '.dependencies["${number}"].version = \""${version}"\"' -i ${chart_dir}/kubeflow/Chart.yaml"
+    ((number = number + 1))
+done