helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Feature Request: Handle updates from `-rcX` to `-rcX+1` in Update chart job

Open viccuad opened this issue 1 year ago • 4 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Current Behavior

On PR https://github.com/kubewarden/helm-charts/pull/220, With job https://github.com/kubewarden/helm-charts/actions/runs/4692840686, Which is a new release for appVersion 1.6.0-rc6, from appVersion 1.6.0-rc5,

The automation incorrectly bumped the minor version of the charts. This may be because the automation doesn't know that we are updating from an RC (rc5 to rc6), so then it bumps the minor version.

The automation incorrectly left appVersion of 1 chart without bumping. This may be because that chart didn't have new changes. (E.g: https://github.com/kubewarden/helm-charts/pull/220/commits/79c35dbf807ae53145df3e5b5c8d939c8759a544).

Expected Behavior

The automation should only bump the rc suffix in version and appVersion.

In addition, the automation should also bump the rc suffix even for those charts that don't contain new changes, so all charts have the same suffix (e.g: -rc6). This means that all the catalog.cattle.io/auto-install will also be up-to-date. (e.g: )

viccuad avatar Apr 14 '23 09:04 viccuad