jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
254 stars 338 forks source link

[jaeger] define securityContext for each resources #527

Open eumel8 opened 6 months ago

eumel8 commented 6 months ago

What this PR does

At least and especially for the all-in-one installation securityContext was missing which brings a bad experienance if you start with Jaeger. This PR

Which issue this PR fixes

Checklist

eumel8 commented 3 months ago

@pavelnikolov no problem. Everyone complains the lack of open source contribution and then it takes months to review a PR. I had this very often and I ask me: What's wrong with my PR's.

pavelnikolov commented 3 months ago

I'm sorry for closing it.