Skip to content

Commit 0f098c3

Browse files
rlengletistio-testing
authored andcommitted
Fix rewriting of istio-cni Helm charts in release scripts (istio#16492)
Rewrite the hub and tag in values_gke.yaml in addition to values.yaml. Remove the "git clone ...cni.git" and values.yaml rewriting from create_charts(), which are redundant with the cloning in create_release_archives.sh and the rewriting in update_helm().
1 parent 52e1b39 commit 0f098c3

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

release/gcb/gcb_lib.sh

+3-9
Original file line numberDiff line numberDiff line change
@@ -180,8 +180,8 @@ function update_helm() {
180180
eval "$unzip_cmd" "${tarball_name}"
181181
rm "${tarball_name}"
182182
# Update version string in yaml files.
183-
sed -i "s|hub: gcr.io/istio-release|hub: ${DOCKER_HUB}|g" ./"istio-${VERSION}"/install/kubernetes/helm/istio*/values.yaml
184-
sed -i "s|tag: .*-latest-daily|tag: ${VERSION}|g" ./"istio-${VERSION}"/install/kubernetes/helm/istio*/values.yaml
183+
sed -i "s|hub: gcr.io/istio-release|hub: ${DOCKER_HUB}|g" ./"istio-${VERSION}"/install/kubernetes/helm/istio*/values.yaml ./"istio-${VERSION}"/install/kubernetes/helm/istio-cni/values_gke.yaml
184+
sed -i "s|tag: .*-latest-daily|tag: ${VERSION}|g" ./"istio-${VERSION}"/install/kubernetes/helm/istio*/values.yaml ./"istio-${VERSION}"/install/kubernetes/helm/istio-cni/values_gke.yaml
185185
current_tag=$(grep "appVersion" ./"istio-${VERSION}"/install/kubernetes/helm/istio/Chart.yaml | cut -d ' ' -f2)
186186
if [ "${current_tag}" != "${VERSION}" ]; then
187187
find . -type f -exec sed -i "s/tag: ${current_tag}/tag: ${VERSION}/g" {} \;
@@ -217,17 +217,11 @@ function create_charts() {
217217
mkdir -vp "${OUTPUT}/istio"
218218
cp -R "./istio-${VERSION}/install" "${OUTPUT}/istio/install"
219219

220-
pushd "$OUTPUT"
221-
git clone -b "${BRANCH}" https://github.com/istio/cni.git
222-
sed -i "s|hub: gcr.io/istio-release|hub: ${DOCKER_HUB}|g" cni/deployments/kubernetes/install/helm/istio-cni/values.yaml
223-
sed -i "s|tag: .*-latest-daily|tag: ${VERSION}|g" cni/deployments/kubernetes/install/helm/istio-cni/values.yaml
224-
popd
225-
226220
# Charts to extract from repos
227221
CHARTS=(
228222
"${OUTPUT}/istio/install/kubernetes/helm/istio"
223+
"${OUTPUT}/istio/install/kubernetes/helm/istio-cni"
229224
"${OUTPUT}/istio/install/kubernetes/helm/istio-init"
230-
"${OUTPUT}/cni/deployments/kubernetes/install/helm/istio-cni"
231225
)
232226

233227
# Prepare helm setup

0 commit comments

Comments
 (0)