jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
265 stars 340 forks source link

adding strategyType support for collector pods #417

Closed swapnilpotnis closed 1 year ago

swapnilpotnis commented 1 year ago

What this PR does

-> This PR adds deployment strategyType of "RollingUpdate" rather than Recreate which is causing downtime whilst at the time of deployment.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Checklist

naseemkullah commented 1 year ago

Thanks @swapnilpotnis but rolling update is the already the default strategy

Did you experience otherwise?

pavelnikolov commented 1 year ago

I agree with @naseemkullah - this already the default behavior. Closing this PR due to inactivity.