8000 [spec] add environment variables as context propagators specification by adrielp · Pull Request #4454 · open-telemetry/opentelemetry-specification · GitHub
[go: up one dir, main page]

Skip to content

Conversation

adrielp
Copy link
Contributor
@adrielp adrielp commented Mar 22, 2025

@adrielp adrielp requested review from a team March 22, 2025 13:52
adrielp and others added 2 commits March 22, 2025 23:15
Co-authored-by: Christophe Kamphaus <christophe.kamphaus@gmail.com>
Copy link
Contributor
@jsuereth jsuereth left a comment

Choose a reason for hiding this comment

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

Overall LGTM.

@adrielp adrielp requested review from jsuereth, reyang and kamphaus March 26, 2025 21:52
Copy link
Member
@reyang reyang left a comment

Choose a reason for hiding this comment

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

LGTM, I have a comment https://github.com/open-telemetry/opentelemetry-specification/pull/4454/files#r2015053822 which can be covered in a follow up PR.

@reyang reyang merged commit e711f74 into open-telemetry:main Apr 1, 2025
6 checks passed
@adrielp adrielp deleted the env-context-spec branch April 2, 2025 06:48
@carlosalberto carlosalberto mentioned this pull request Apr 8, 2025
carlosalberto added a commit that referenced this pull request Apr 15, 2025
### Context

- Add context propagation through Environment Variables specification.

([#4454](#4454))
- On Propagators API, stabilize `GetAll` on the `TextMap` Extract.

([#4472](#4472))

### Traces

- Define sampling threshold field in OpenTelemetry TraceState; define
the behavior
of TraceIdRatioBased sampler in terms of W3C Trace Context Level 2
randomness.

([#4166](#4166))

### Metrics

- Clarify SDK behavior for Instrument Advisory Parameter.

([#4389](#4389))

### Logs

- Add `Enabled` opt-in operation to the `LogRecordProcessor`.

([#4439](#4439))
- Stabilize `Logger.Enabled`.

([#4463](#4463))
- Stabilize `EventName`.

([#4475](#4475))

### Baggage

- Add context (baggage) propagation through Environment Variables
specification.

([#4454](#4454))

### Resource

- Add Datamodel for Entities.

([#4442](#4442))

### SDK Configuration

- Convert declarative config env var substitution syntax to ABNF.

([#4448](#4448))
- List declarative config supported SDK extension plugin interfaces.

([#4452](#4452))

---------

Co-authored-by: Armin Ruech <7052238+arminru@users.noreply.github.com>
zerok added a commit to grafana/tanka that referenced this pull request Aug 28, 2025
With this it should be possible to keep track of performance issues also
when called from wrapper tools.

This tries to adhere to the environment specification laid out in
open-telemetry/opentelemetry-specification#4454
github-merge-queue bot pushed a commit to grafana/tanka that referenced this pull request Aug 29, 2025
* feat: Add OpenTelemetry tracing support

With this it should be possible to keep track of performance issues also
when called from wrapper tools.

This tries to adhere to the environment specification laid out in
open-telemetry/opentelemetry-specification#4454

* Add spans to core paths

* Add docs

* Remove too noisy span on jsonnet.Evaluate

* Remove spans on the loader level to reduce noise

* Reduce noise further

* Improve consistency
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.

Add specification for Env propagation leveraging prototypes
5 participants
0