Skip to content

Conversation

adrielp
Copy link

@adrielp adrielp commented Jun 2, 2025

Description

Adds a prototype for the environment variable carrier, specifically decorating Getter and Setter instead of creating a dedicated propagator. The comes from the conversations within the environment variable context propagation specification pull request.

There's a corresponding thread in Slack around this topic.

The example use of the Getter portion of this prototype can be found in this gist.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

The Getter was tested by using the prototype in a real environment. Unit tests need to be added after conversation around this prototype.

  • Test A

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.
  • Unsure

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

github-merge-queue bot pushed a commit to open-telemetry/opentelemetry-specification that referenced this pull request Jul 8, 2025
## Changes

Adds supplementary guidance around the behaviors the SDKs should look to
implement.

* [x] Related issues
[#4470](#4470)
& #4484
* [x] Related [OTEP
0258](https://github.com/open-telemetry/opentelemetry-specification/blob/main/oteps/0258-env-context-baggage-carriers.md)
* [x] Links to the prototypes (when adding or changing features)
  - [Go](open-telemetry/opentelemetry-go#6778)
-
[Python](open-telemetry/opentelemetry-python#4609)
* [x]
[`CHANGELOG.md`](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CHANGELOG.md)
file updated for non-trivial changes

---------

Co-authored-by: Robert Pająk <pellared@hotmail.com>
Co-authored-by: Liudmila Molkova <limolkova@microsoft.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@adrielp adrielp force-pushed the env-carrier-proto branch from 259041f to ccdb1d8 Compare August 31, 2025 23:12
@adrielp adrielp marked this pull request as ready for review September 1, 2025 01:46
@adrielp adrielp requested a review from pellared September 1, 2025 01:48
@xrmx xrmx moved this to Ready for review in @xrmx's Python PR digest Sep 9, 2025
@StephenWithPH StephenWithPH mentioned this pull request Sep 16, 2025
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for review
Development

Successfully merging this pull request may close these issues.

2 participants