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

Update semantic-conventions to 1.21.0, publish all schemas, drop draft status, enable auto-update #3033

Merged
merged 1 commit into from
Jul 14, 2023

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jul 14, 2023

  • Contributes to Adjust to move Semantic Conventions out of OTel spec #2721
  • Updates semconv submodule to 1.21.0
  • Enabled the publication of schema for 1.21.0
  • Removes draft status from semconv pages -- SEMCONV PAGES WILL APPEAR ON THE PRODUCTION SERVER
  • Enabled auto-update of the semconv version
  • Note that the OTel spec is at 1.23.0, so links into the OTel spec at 1.22.0 appear as external links; they do not link to the corresponding OTel website spec page

Preview: https://deploy-preview-3033--opentelemetry.netlify.app/docs/specs/semconv

/cc @jsuereth @joaopgrassi

@chalin chalin requested review from a team July 14, 2023 01:29
@joaopgrassi
Copy link
Member

joaopgrassi commented Jul 14, 2023

Does Rename pages under /docs/general/ to avoid general in the page name, etc impact in anything after we release this version to production? I guess the URLs right, but that can be fixed with the aliases?

@chalin
Copy link
Contributor Author

chalin commented Jul 14, 2023

@joaopgrassi - we can do it in either order. Fixing the names first is a bit easier in that we don't need to add aliases.

If you and @jsuereth prefer, I can leave the senconv pages in draft status. Let me know.

@jsuereth
Copy link

I think publishing as-is is fine. We can continue to refine/update pages as we go.

@chalin chalin merged commit 9f8e1aa into open-telemetry:main Jul 14, 2023
@chalin chalin deleted the chalin-im-semconv-1.21.0 branch July 14, 2023 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants