search

open-telemetry / opentelemetry-go

OpenTelemetry Go API and SDK
https://opentelemetry.io/
Apache License 2.0
5.16k stars 1.04k forks source link

Migrate from deprecated semconvgen to weaver #5824

Closed dashpole closed 17 hours ago

dashpole commented 2 days ago

Problem Statement

The OTel community is consolidating on using weaver for generating semantic conventions in language SDKs.

We should migrate from https://pkg.go.dev/go.opentelemetry.io/build-tools/semconvgen to https://github.com/open-telemetry/weaver.

Prior Art

PoC: https://github.com/open-telemetry/opentelemetry-go/pull/5793

dashpole commented 2 days ago

@jsuereth please add any important details here. Is this blocking support for semconv v1.27.0?

dashpole commented 2 days ago

Part of https://github.com/open-telemetry/weaver/issues/227

jsuereth commented 2 days ago

It's not blocking support of 1.27.0, but some kind of handling of 1.27.0 is needed. You can either migrate to weaver now or invest in improving your existing jinja templates now, and still break later.

dmathieu commented 18 hours ago

Is this a duplicate of #5668?

dashpole commented 17 hours ago

Is this a duplicate of https://github.com/open-telemetry/opentelemetry-go/issues/5668?

Ah, I didn't find that. Closing in favor of #5668