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.

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

Proposed semconv pre-release process #1817

Closed
chalin opened this issue Jan 24, 2025 · 4 comments
Closed

Proposed semconv pre-release process #1817

chalin opened this issue Jan 24, 2025 · 4 comments

Comments

@chalin
Copy link
Contributor

chalin commented Jan 24, 2025

@trask - great work on getting the 1.30.0 release out! 🙌🏻 🎉

I think that open-telemetry/opentelemetry.io#6077 is somewhat typical of new semconv release PRs. Inevitably issues arise, such as the following, and we end up doing double the work (fixing semconv and patching OTel.io):

@trask and I have been working on improving the release process. @trask created a daily build script. That allowed us to catch some issues ahead of the new release, such as:

I think that the best way forward is to open a PR over OTel.io and incrementally (maybe daily) update the semconv repo version used by that PR. If any issues arise, then an @open-telemetry/docs-maintainers can fix them and push changes to the PR. The PR also give us the ability to preview the semconv release.

(That's what I did with open-telemetry/opentelemetry.io#6060, which built without any issues. The problem is that more updates were applied to the semconv repo, and not retested.)

Thoughts @trask @lmolkova et all?

@trask
Copy link
Member

trask commented Jan 24, 2025

ya, that sounds like a better way to go 👍

I'm going to start working on some automation for it

@chalin
Copy link
Contributor Author

chalin commented Jan 31, 2025

Seems to be working very well atm. ❤ Thanks for putting this together! 🙌🏻

@trask
Copy link
Member

trask commented Feb 1, 2025

thanks for all your help!

anything remaining you know of, or should we close this and see how things go?

@chalin
Copy link
Contributor Author

chalin commented Feb 1, 2025

Let's close and see how things go.

@chalin chalin closed this as completed Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Need triage
Development

No branches or pull requests

2 participants