jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
266 stars 343 forks source link

[jaeger-operator] update to 1.34.1 #367

Closed czomo closed 2 years ago

czomo commented 2 years ago

Signed-off-by: czomo tomaszjdul@gmail.com

What this PR does

Updates jaeger-operator update to 1.34.1

Which issue this PR fixes

Checklist

czomo commented 2 years ago

/hold I noticed we missing something big. Looking at https://github.com/jaegertracing/jaeger-operator/pull/1864/files, jaeger-operator introduced another dependency. That's why multiple people complaining about complaining about missing certs(since 2.30.0 of this chart using appVersion:v1.31). There is no certificate manifest and so on, example https://github.com/jaegertracing/jaeger-operator/blob/faa5c70a09abc98170b9e175e76d72ef9a1764af/config/certmanager/certificate.yaml.

@cpanato Should we create separate PR or add it here? Wdyt?

czomo commented 2 years ago

https://github.com/jaegertracing/helm-charts/pull/368 is merged into main. Now we can bump the image FYI @cpanato @batazor