Skip to content

Commit d8a0e8b

Browse files
authored
Merge pull request #2775 from ConnorJC3/fix-patch-release-script
Fix patch releases in generate-release-pr script
2 parents fbbc758 + 46c85e6 commit d8a0e8b

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

hack/release-scripts/generate-release-pr

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,10 +112,17 @@ update_installmd() {
112112

113113
update_chart_and_overlays() {
114114
log "Updating helm chart and generates kustomize"
115-
prev_minor_patch_version=$(echo "$PREV_DRIVER_VERSION" | sed 's/v[0-9]*\.//')
116-
new_minor_patch_version=$(echo "$NEW_DRIVER_VERSION" | sed 's/v[0-9]*\.//')
115+
new_driver_version_no_prefix=$(echo "$NEW_DRIVER_VERSION" | sed 's/v//')
117116

118-
$SED "s/$prev_minor_patch_version$/$new_minor_patch_version/g" -i "$CHART_PATH"
117+
# Set appVersion to new driver version
118+
yq ".appVersion=\"$new_driver_version_no_prefix\"" -i "$CHART_PATH"
119+
# If minor version has changed, set chart version to appVersion with 2 as major version
120+
# If not (patch release), just bump the chart patch version
121+
if [ "${PREV_DRIVER_VERSION%.*}" != "${NEW_DRIVER_VERSION%.*}" ]; then
122+
yq ".version = (\"$new_driver_version_no_prefix\" | split(\".\") | .[0] = \"2\" | join(\".\"))" -i "$CHART_PATH"
123+
else
124+
yq '.version |= (split(".") | .[2] |= (to_number + 1) | join("."))' -i "$CHART_PATH"
125+
fi
119126

120127
(
121128
cd "$ROOT_DIRECTORY"

0 commit comments

Comments
 (0)