Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Migrate from deprecated semconvgen to weaver #5824

Closed
dashpole opened this issue Sep 16, 2024 · 5 comments
Closed

Migrate from deprecated semconvgen to weaver #5824

dashpole opened this issue Sep 16, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@dashpole
Copy link
Contributor

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: #5793
@dashpole dashpole added the enhancement New feature or request label Sep 16, 2024
@dashpole
Copy link
Contributor Author

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

@dashpole
Copy link
Contributor Author

Part of open-telemetry/weaver#227

@jsuereth
Copy link

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
Copy link
Member

Is this a duplicate of #5668?

@dashpole
Copy link
Contributor Author

Is this a duplicate of #5668?

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dmathieu @jsuereth @dashpole