resource: add WithService detector option

[codeboten]

I was looking at implementing resource detection in otelconf and was finding that all the detectors were implemented in a similar way except for the service detector. Added the resource options in this PR for feedback. Will add tests if the go approvers/maintainers support this approach.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.1%. Comparing base (a72103c) to head (841fd62).

Files with missing lines	Patch %	Lines
sdk/resource/config.go	0.0%	9 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##            main   #7642     +/-   ##
=======================================
- Coverage   86.1%   86.1%   -0.1%     
=======================================
  Files        298     298             
  Lines      21694   21703      +9     
=======================================
  Hits       18692   18692             
- Misses      2626    2635      +9     
  Partials     376     376             

Files with missing lines	Coverage Δ
sdk/resource/config.go	87.6% <0.0%> (-12.4%)	⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[dmathieu]

This approach looks good to me. Adding tests would be awesome (there are no tests for the rest of the config unfortunately).

[dashpole]

We currently have this behind an experimental feature gate in the SDK:

opentelemetry-go/sdk/resource/resource.go

Lines 242 to 243 in 729bd6e

	if x.Resource.Enabled() {
	defaultDetectors = append([]Detector{defaultServiceInstanceIDDetector{}}, defaultDetectors...)

It looks like service.instance.id is still marked as being in development, so i'm not sure we can add a public API for the detector yet: https://github.com/open-telemetry/semantic-conventions/blob/main/docs/registry/attributes/service.md#service-attributes

[codeboten]

There is a public API for WithContainerID and that is still in development https://github.com/open-telemetry/semantic-conventions/blob/main/docs/registry/attributes/container.md, should the api be removed?

[dashpole]

I would not have added it. I don't think we can remove it now, though.