Tyler Yahn
Tyler Yahn
@josepedropereira @pellared should this PR be closed? What was the resolution of [this](https://github.com/open-telemetry/opentelemetry-go-contrib/pull/5431#pullrequestreview-2024891968)?
This seems duplicative of [`WithHost`](https://pkg.go.dev/go.opentelemetry.io/otel/[email protected]/resource#WithHost). Why not update that option?
Taking a look at the host semantic conventions, it looks like they are all experimental (even `host.name` :grimacing:). I think in the long term we want to make the `WithHost`...
> > But in the interim, having a detector here is probably the correct approach. > > @MrAlias Should we use feature environment variables instead of a new detector module?...
> > > But in the interim, having a detector here is probably the correct approach. > > > > > > @MrAlias Should we use feature environment variables instead...
@pyohannes can this be closed? We plan to move this into [this](https://github.com/open-telemetry/opentelemetry-go/blob/6d45f283c7abf2ee5284c769f9ae7109096d3564/sdk/resource/host_id.go) (behind the experimental envar), right?
Kick CI.
Client semconv template work is still blocked: https://github.com/open-telemetry/opentelemetry-go-contrib/pull/5773
We still need resolution for #6261 and #6262 to have a complete reference implementation to templatize.