jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
258 stars 340 forks source link

[jaeger] Updating kafka chart dependency back to 19.x due to breaking change introduced in new kafka tag #476

Closed mvnarsing closed 1 year ago

mvnarsing commented 1 year ago

What this PR does

Reverting kafka chart updates While testing Jaeger with new kafka chart we found kafka has new bug introduced with this chart which is crashing kafka. Known bug: https://github.com/bitnami/containers/pull/34152

Sorry for confusion, didn't see this new bug until we upgraded and tested in our environment

Which issue this PR fixes

Reverting kafka chart so kafka works with old bitnami images

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Checklist

pavelnikolov commented 1 year ago

@mvnarsing the commits need to be signed

pavelnikolov commented 1 year ago

Closing the PR as the commit was not signed. Fixed with this commit.