Naseem

Results 109 comments of Naseem

Hi @msschl we can set some for the various components' securityContext values. https://github.com/jaegertracing/helm-charts/issues/41 is an issue open to decide what we should do, running all containers as non root is...

Alright, I think in this case we should test and then add default securityContext for every pod via the various values https://github.com/jaegertracing/helm-charts/blob/d4042b2affa940d61ba7aee3c755adfae6a6d0f7/charts/jaeger/values.yaml#L233 to runAsNonRoot and runAsUser: 1000

Great! I think you can get by with just specifying one of the two btw.

On another note I think it's time we set runAsNonRoot as default for all components... with say user `1000`, related issue: https://github.com/jaegertracing/helm-charts/issues/41

[Please GPG sign your commits.](https://docs.github.com/articles/about-gpg/)

As per https://github.com/jaegertracing/helm-charts/tree/main/charts/jaeger#elasticsearch-rollover, ES should be deployed separately from the jaeger chart (not as a dependency) for this to work with the current limitation. Maybe that could be reworded more...

Hi @paulpuvi06 https://github.com/jaegertracing/jaeger/issues/2366 has been created on your behalf please check for a response from @jaegertracing/jaeger-maintainers

HI @paulpuvi06 as per @pavolloffay's reponse in jaegertracing/jaeger#2366, the port should be appended to `CQLSH_HOST`. Please let me know if that works for you.

> - name: CQLSH_HOST > value: tracing.cassandra.cosmos.azure.com** It appears that the second `CQLSH_HOST` is overriding the first, please only have it set once.

Hi @paulpuvi06 I see the limitation that is causing your issue, could you please open a PR to address it? https://github.com/jaegertracing/helm-charts/blob/32d7fb792a66b7977be649e52d607bda5e8d25c2/charts/jaeger/templates/cassandra-schema-job.yaml#L47 Should also include port, in particular if it is...