Gary Brown

Ok thanks - we will try to reproduce.

@tux-o-matic I think this sounds like a separate issue - nothing to do with sidecar injected agent flags not being updated on upgrade. Could you create a separate issue?

@jpkrohling Thoughts? I think the control over the volume/mounts was to prevent definitions at the top level inadvertently being added to the app deployments, however as @Saad-Hussain1 says, if the...

@abhidha02 I think this is a helm chart issue, so have moved to that repo. If you think this is wrong, let me know.

@subnetmarco could you also apply the same fix to the [production template](https://github.com/jaegertracing/jaeger-kubernetes/blob/master/jaeger-production-template.yml)?

Hi @subnetmarco can you also address the points raised by Pavol in https://github.com/jaegertracing/jaeger-kubernetes/pull/132#issuecomment-608287318

@pavolloffay I created the issue as @cmoulliard had asked about how to obtain the tracer. So depends whether he thinks this needs documenting? Otherwise can be closed.

I would also vote for 1.

Agree, option 1 is fine for now.

+1 for withSpan