search

open-telemetry / opentelemetry-operator

Kubernetes Operator for OpenTelemetry Collector
Apache License 2.0
1.2k stars 438 forks source link

Support for OpenTelemetryCollector `readinessProbe` #2943

Closed janario closed 5 months ago

janario commented 6 months ago

Component(s)

collector

Is your feature request related to a problem? Please describe.

We were testing high-availability of the collector (managed by the operator) while we roll out changes and we noticed that the Pods created by the operator don't have readinessProbe configured.

Describe the solution you'd like

Support readinessProbe for the collector pods using the default as provided by the chart https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/values.yaml#L397

Describe alternatives you've considered

No response

Additional context

We could achieve high-availability while rolling out with readinessProbe(to be addressed here) and some sleep at the preStop (really custom image, to be addressed at the collector repo later).

janario commented 6 months ago

We should have a PR for this one by tomorrow :-)