zookeeper-operator icon indicating copy to clipboard operation
zookeeper-operator copied to clipboard

Latest tag is missing cluster and operator chart version bump

Open jravetch opened this issue 3 years ago • 8 comments

Description

Tag 0.2.13 is missing commit https://github.com/pravega/zookeeper-operator/commit/4ceccde6cdc60c055fd35e328ef88100e73551cf that bumps the cluster and operator charts. Currently tag 0.2.13 deploys 0.2.12 of the charts

Importance

Blocker

Location

Please see https://github.com/pravega/zookeeper-operator/commit/4ceccde6cdc60c055fd35e328ef88100e73551cf

Suggestions for an improvement

Create a new tag that matches the cluster and operator chart version

jravetch avatar Mar 31 '22 21:03 jravetch

@jravetch -- I think this is a technicality of how the release needs to managed in conjunction with https://github.com/pravega/charts

Maybe @anishakj or @SrishT can explain further.

How do you consume this repo? I can see how what you're talking about could be an issue for certain consumers of this repo, depending on how you're taking in releases.

derekm avatar Apr 05 '22 17:04 derekm

@jravetch If you want to install 0.2.13 version of zookeeper, you can use https://github.com/pravega/charts.

helm repo add pravega https://charts.pravega.io
helm repo update
helm install [RELEASE_NAME] pravega/zookeeper-operator --version=0.2.13

anishakj avatar Apr 08 '22 14:04 anishakj

@anishakj @derekm That's exactly what we are doing, but the pod template spec is still using 0.2.12 version of the image tag

Containers:                                                                                         
  zookeeper:                   
    Container ID:  docker://ba871cdb9173352506875bf5dde4d69d6efcfb823e310a84ec38fbccbc7ffafb
    Image:         pravega/zookeeper:0.2.12
helm list
zookeeper-operator      zookeeper-operator              4               2022-04-02 00:19:15.294388606 +0000 UTC         deployed        zookeeper-operator-0.2.13       0.2.13

jravetch avatar Apr 11 '22 22:04 jravetch

We are using helmfile sync to install the chart

jravetch avatar Apr 11 '22 23:04 jravetch

@jravetch we are seeing 0.2.13 with helm install

zookeeper:
    Container ID:  containerd://b37753a98a22239215a10afc4a1373723371a458b9118ad383ae3454c9768997
    Image:         pravega/zookeeper:0.2.13

anishakj avatar Apr 18 '22 10:04 anishakj

Is this a fresh install or an upgrade?

jravetch avatar Apr 18 '22 16:04 jravetch

Is this a fresh install or an upgrade?

fresh install

anishakj avatar Apr 18 '22 17:04 anishakj

Can you test installing 0.2.12 and upgrading to 0.2.13?

jravetch avatar Apr 18 '22 18:04 jravetch