Tyler Helmuth
Tyler Helmuth
Actually this issue is still blocked by https://github.com/open-telemetry/opentelemetry-collector/issues/7111
I believe issues are being tracked in https://github.com/open-telemetry/opentelemetry-collector/milestone/31
@Sanket-0510 try doing a `make install-tools` and then a `make gotidy`
All tests are now passing except Contrib, which I'll fixed after this is merged should we choose to go forward with this breaking change approach.
The origin of the `Experiment` designation is [here](https://github.com/open-telemetry/opentelemetry-collector/pull/4547#discussion_r779812787). On one hand it has been 2 years and requests to include only certain headers has not surfaced. The feature has been...
Related: client lists [`client.Info.Metadata`](https://github.com/open-telemetry/opentelemetry-collector/blob/4ae8d2bcd5ae2e72280fb6af7e9402b7b6e7c6cb/client/client.go#L100-L102) as experimental from the same PR. Whether or not we stick with the `IncludeMetadata` config interface I think `client.Info.Metadata` is here to stay. I think we...
@arjunmahishi please take a look at the conflict.
@dmitryax it feels pretty natural from an ergonomic perspective within our components to have a single config struct and and then be able to call `ToServer` on it to get...
I've updated the list in https://github.com/open-telemetry/opentelemetry-collector/issues/9375 to reflect the dependency order mentioned here. Closing as completed
> We would need to make client into a module on the first place 😄 I've been bitten by our template lol I want to be able to call this...