Skip to content

feat: rest-client HTTP semantic convention stability migration#1568

Merged
kaylareopelle merged 8 commits intoopen-telemetry:mainfrom
hannahramadan:restclient_semconv_stability_migration
Jul 28, 2025
Merged

feat: rest-client HTTP semantic convention stability migration#1568
kaylareopelle merged 8 commits intoopen-telemetry:mainfrom
hannahramadan:restclient_semconv_stability_migration

Conversation

@hannahramadan
Copy link
Copy Markdown
Member

@hannahramadan hannahramadan commented Jun 18, 2025

This PR is intended to assist in the transition from the old to new HTTP semantic conventions. Per the HTTP semantic convention stability migration spec, users should be able to set the environment variable OTEL_SEMCONV_STABILITY_OPT_IN to:

  • http to emit stable conventions only
  • http/dup to emit both old and the stable conventions
  • a blank env var will emit old conventions only
  • Span names have also changed. Spans will now be named based on only their method. For example, what used to be HTTP GET is now just GET - spec

The agent is required to maintain this bridge for 6 months and may drop the environment variable in the next major version and emit only the stable HTTP and networking conventions.

This approach was approved in #1547

@hannahramadan hannahramadan marked this pull request as draft June 20, 2025 23:10
@hannahramadan hannahramadan marked this pull request as ready for review June 23, 2025 18:04
Copy link
Copy Markdown
Contributor

@kaylareopelle kaylareopelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo in the README, other than that this is good to go!

Comment thread instrumentation/restclient/README.md Outdated
Comment thread instrumentation/restclient/README.md Outdated
Copy link
Copy Markdown
Contributor

@ericmustin ericmustin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some small readme typos but nothing blocking

hannahramadan and others added 2 commits July 28, 2025 11:21
Co-authored-by: Kayla Reopelle <87386821+kaylareopelle@users.noreply.github.com>
@kaylareopelle kaylareopelle enabled auto-merge (squash) July 28, 2025 18:22
@kaylareopelle kaylareopelle merged commit 98446e7 into open-telemetry:main Jul 28, 2025
62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants