Naseem

I agree with both those statements, though for the second if possible keep as is and allow user to override the array of ports, in yaml since you cant concat...

> `for the second if possible keep as is and allow user to override the array of ports, in yaml since you cant concat arrays` > Can you clarify with...

cc @cpanato @batazor can you please have a look?

cc @cpanato @batazor

You want to deploy your Jaeger-Operator managed Jaegers(CRDs) through a helm chart, is that it? I do not use the jaeger operator currently so am unqualified to answer. @cpanato @batazor...

> Do I need to bump chart version or update README.md? Just version bump please, if not CI will 💥 if I'm not mistaken

Deleting the deployment with `--cascade=false` will save you from downtime as well (provided you reinstall immediately after).

First approach would be outside the scope of this chart, and probably not recommended. Second approach can be achieved as you mentioned but requires adding the container as it was...

Thanks @yurishkuro I will do this that way then.

Thanks for raising this issue @hardik-fareye, we've just released 0.46.6 where @cwrau imported bitnami's https://github.com/bitnami/charts/tree/master/bitnami/common to standardize on api versioning logic, could you please try it out, if it still...