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

[jaeger-operator] Helm Chart incorrectly has appVersion 1.31.0

Open mjnagel opened this issue 2 years ago • 4 comments

Describe the bug https://github.com/jaegertracing/helm-charts/commit/d7671badf15944ad4721d34f663e444e42bb2511 bumped the chart version and appVersion but did not actually update to the latest CRD and image for the operator.

Going a step further, even overriding the helm values to use the 1.31.0 image will result in errors due to the introduction of cert management in https://github.com/jaegertracing/jaeger-operator/pull/1708

To Reproduce Steps to reproduce the behavior:

  1. Deploy the latest helm chart
  2. Validate that the 1.30.0 image is being used despite appVersion of 1.31.0

Expected behavior I would expect 1.31.0 to be deployed based on the appVersion.

mjnagel avatar Mar 08 '22 20:03 mjnagel

my bad, thanks for noticing

kinda explains why i didn't run into any issues with the upgrade 😅

hairmare avatar Mar 10 '22 00:03 hairmare

We've also run across this issue, after updating to 1.31 or 1.32 jaeger-operator fails.

eanton86 avatar Mar 17 '22 04:03 eanton86

I faced the same. The last working version is 1.30

sta-szek avatar Mar 17 '22 11:03 sta-szek

Issues when overriding the image are the same as reported in https://github.com/jaegertracing/helm-charts/issues/351 . There seems to be a lack of quality reviewing of updates...CRDs are rarely updated and it seems like the changes are not even tested based on the cert manager issues.

mjnagel avatar Apr 26 '22 20:04 mjnagel