Support for collector readinessProbe

[janario]

Description:

This PR introduces a possibility to customize readinessProbe for OpenTelemetryCollector as well as adds the default just like it is done at liveness and collector chart https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/values.yaml#L397

Link to tracking Issue(s): #2943

• Resolves: Support for OpenTelemetryCollector readinessProbe #2943

Testing:

Documentation:

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: hesamhamdarsi (166338f, f839847)
• ✅ login: janario / name: Janario Oliveira (88fcb72, 03008af, dbeb0ce, 270cc6d, 6f19ccb, 7c61f89, 9835ddd, 800bf85)

[swiatekm-sumo]

I wouldn't add this to v1alpha1. That version is now deprecated, and we shouldn't add any more features to it. WDYT @pavolloffay @jaronoff97

[jaronoff97]

+1 we should only add this to v1beta1.

[janario]

Got it!
Removed from alpha version.

Just one detail to see if I got it right.

Deploying alpha won't have a way to configure anything about readiness but it will enable if healthcheck extension is available.

In case the user needs to customize it, they will required to update to beta.

Right?

[swiatekm-sumo]

LGTM, with one nitpick.

[swiatekm-sumo]

Deploying alpha won't have a way to configure anything about readiness but it will enable if healthcheck extension is available.

In case the user needs to customize it, they will required to update to beta.

Right?

Yes, that's correct. I think it's fine, especially with a change as safe as this.

[janario]

Thank you all for the review ;-)

Looking forward to get this and a few other improvements in the next releases 💪 ;-)