Add path option for health check extension

[cuichenli]

Description: Add the ability to specify the path health check should serve.

Link to tracking Issue: Fix open-telemetry/opentelemetry-collector#3002

Testing: A few unit tests were added.

[linux-foundation-easycla]

The committers are authorized under a signed CLA.

• ✅ Will Li (98e60e2, 2e55c90, b0749f3, 5b5ca8c, e78d440)

[MovieStoreGuy]

This all seems rather reasonable, just a few nits that are not impacting operations.

[MovieStoreGuy]

Groovy, thanks for doing the work.

We just need a maintainer to run the tests :)

Could I bother either @tigrannajaryan or @jpkrohling to run them for you :)

[jpkrohling]

Tests started.

[tigrannajaryan]

healthcheckextension_test.go:230: File is not `gofmt`-ed with `-s` (gofmt)

You should be able to run the tests locally, no need to wait for maintainers and for CI. Make sure make passes locally before creating a PR, see https://github.com/open-telemetry/opentelemetry-collector/blob/main/CONTRIBUTING.md#creating-a-pr

[cuichenli]

healthcheckextension_test.go:230: File is not `gofmt`-ed with `-s` (gofmt)

You should be able to run the tests locally, no need to wait for maintainers and for CI. Make sure make passes locally before creating a PR, see open-telemetry/opentelemetry-collector@main/CONTRIBUTING.md#creating-a-pr

thanks, did not notice i can do that...

i have fmt the code and can either @tigrannajaryan or @jpkrohling run the test again? 🙇‍♂️

[jpkrohling]

Tests started.

[cuichenli]

some tests failed somehow, not sure if they are flaky tests?

[jpkrohling]

Indeed, this is getting affected by #5752.

[jpkrohling]

@VineethReddy02, would you like to review this one?

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 7 days.

[VineethReddy02]

LGTM!