Skip to content

feat(semantic-conventions): update semantic conventions to v1.33.0#5654

Merged
trentm merged 1 commit intoopen-telemetry:mainfrom
trentm:trentm-semconv-1.33.0
May 7, 2025
Merged

feat(semantic-conventions): update semantic conventions to v1.33.0#5654
trentm merged 1 commit intoopen-telemetry:mainfrom
trentm:trentm-semconv-1.33.0

Conversation

@trentm
Copy link
Copy Markdown
Contributor

@trentm trentm commented May 6, 2025

Summary of semconv changes in the changelog here: https://github.com/open-telemetry/opentelemetry-js/pull/5654/files#diff-9f239f4331d2856552d1cfd71fb5585d24d9ef409c7cbcce2e0a59a76e631278R12

Highlights: DB semantics are stable; also some stabilized code.* constants for stacktraces.

@trentm trentm self-assigned this May 6, 2025
@trentm trentm requested a review from a team as a code owner May 6, 2025 23:52
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.03%. Comparing base (60d3115) to head (7fc42dd).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5654   +/-   ##
=======================================
  Coverage   95.03%   95.03%           
=======================================
  Files         310      310           
  Lines        7952     7952           
  Branches     1605     1605           
=======================================
  Hits         7557     7557           
  Misses        395      395           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trentm
Copy link
Copy Markdown
Contributor Author

trentm commented May 7, 2025

Lint failure is:

> opentelemetry@0.1.0 docs:test
> linkinator docs --silent --retry && linkinator doc/*.md --skip http://localhost:3000 --skip http://localhost:9464 --silent --retry --directory-listing

🏊‍♂️ crawling docs
🤖 Successfully scanned 53 links in 0.341 seconds.
🏊‍♂️ crawling doc/context.md doc/esm-support.md doc/exporter-guide.md doc/frequently-asked-questions.md doc/instrumentation-guide.md doc/metrics.md doc/propagation.md doc/sdk-registration.md doc/tracing.md doc/upgrade-to-2.x.md
[429] https://github.com/prometheus/OpenMetrics/blob/v1.0.0/specification/OpenMetrics.md
[429] https://github.com/open-telemetry/opentelemetry-js/blob/main/experimental/packages/opentelemetry-instrumentation-xml-http-request/src/xhr.ts
...

this is GH throwing 429s. I've noticed this twice today on OTel JS PRs and recently from similar doc link checking in other repos. If this becomes a consistent problem, I can work on a fix/workaround. In another repo I worked on the "fix" was basically to ignore 429s (with some caching to attempt to avoid re-checking all links every time).

@trentm trentm added this pull request to the merge queue May 7, 2025
Merged via the queue into open-telemetry:main with commit eeec126 May 7, 2025
18 of 20 checks passed
@trentm trentm deleted the trentm-semconv-1.33.0 branch May 7, 2025 15:48
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this pull request Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants