Robert Pająk

Results 392 comments of Robert Pająk

https://github.com/open-telemetry/opentelemetry-go/blob/d423bd4cf28f65bdcbd1b1d83d0f59c4388f37b0/README.md#L23-L24 You can help with existing TODO issues related to metrics or help addressing existing bugs.

> What about managed duplication through code generation? That is exactly what I propose [here](https://github.com/open-telemetry/opentelemetry-go/issues/3548#issuecomment-1443172592) > We could address the problem e.g. by using "code generation" to "copy/paste" the common...

Closing. I assigned https://github.com/open-telemetry/opentelemetry-go/issues/3548 to myself.

> I think the situation is more complicated than described in this documentation update. It isn't the case that modules are only compatible at the same version (`go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.11.1` is...

> I'm not comfortable with the consolidation on an internal package in the API module with dependence on that module by the SDK modules, as discussed [here](https://github.com/open-telemetry/opentelemetry-go/pull/3806#pullrequestreview-1315949613). I think we...

`otelhttp` no longer uses `httpconv`. You can make a PR directly in https://github.com/open-telemetry/opentelemetry-go-contrib. Sorry for late feedback.

Closing as duplicate of https://github.com/open-telemetry/opentelemetry-go/issues/2341

According the the spec, for gRPC protocol, neither `WithEndpoint` nor `OTEL_EXPORTER_OTLP_ENDPOINT` env var requires providing the scheme. See https://github.com/open-telemetry/opentelemetry-specification/blob/v1.20.0/specification/protocol/exporter.md > **Insecure**: Whether to enable client transport security for the exporter's...

I am favor of not doing any breaking change. I find [the existing documentation](https://pkg.go.dev/go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp#WithEndpoint) is good enough: > Note that the endpoint must not contain any URL path. We could...

SIG meeting: We can add `WithEndpointURL` and deprecate `WithEndpoint` in `otlptracehttp`. Before `otlpmetrichttp` is stable we can add `WithEndpointURL` and remove `WithEndopoint` from `otlpmetrichttp`. However, we may also release stable...