add host.id resource provider

[zeitlinger]

this detector is meant for bare metal - if no cloud provider host detector was found

[breedx-splk]

Looks good, had a couple small asks. I might have split this into two implementations -- one windows specific and one linux specific, and had the HostIdResourceProvider delegate to them depending on the os type, but this is fine.

[zeitlinger]

@laurit can you check?

[laurit]

Could you also update https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/resources/library/README.md

[laurit]

@breedx-splk could you review

[laurit]

@breedx-splk you are blocking this PR from being merged, please review

[breedx-splk]

Hey @zeitlinger I left a PR against this PR to show you what I was talking about by removing the enum.
zeitlinger#1 👍🏻

Seems fine mostly.

[laurit]

@zeitlinger could you also update https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/resources/library/README.md Note that the existing doc is a bit inconsistent, it uses Provider for some and Implementation for other. I think it should actually point to the provider class instead of the resource as provider should be the one that is used to disable it (isn't it?).

[zeitlinger]

@zeitlinger could you also update https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/resources/library/README.md Note that the existing doc is a bit inconsistent, it uses Provider for some and Implementation for other. I think it should actually point to the provider class instead of the resource as provider should be the one that is used to disable it (isn't it?).

This page is also missing other resource providers. WDYT about adding a note to open-telemetry/opentelemetry.io#4027 to document resource providers on telemetry.io?

[laurit]

This page is also missing other resource providers. WDYT about adding a note to open-telemetry/opentelemetry.io#4027 to document resource providers on telemetry.io?

The only one that is missing is JarServiceNameDetector. Documenting on opentelemetry.io is going to be tricky , before that happens imo it would make sense to continue doing things as we have done so far.

[zeitlinger]

This page is also missing other resource providers. WDYT about adding a note to open-telemetry/opentelemetry.io#4027 to document resource providers on telemetry.io?

The only one that is missing is JarServiceNameDetector. Documenting on opentelemetry.io is going to be tricky , before that happens imo it would make sense to continue doing things as we have done so far.

OK, added 😄