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.

Sign up for GitHub

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

Context propagation to client instrumentation #3825

Closed
wants to merge 22 commits into from
+161 −9
Closed

Context propagation to client instrumentation #3825

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
29e2422
Context propagation to client instrumentation
jpkrohling Jan 15, 2024
c411f07
update pr number
jpkrohling Jan 15, 2024
abd5335
markdown-top
jpkrohling Jan 15, 2024
d8fe15f
s/SDKs/instrumentation libraries
jpkrohling Jan 15, 2024
b3d7d92
s/clientpropagator/responsepropagator/
jpkrohling Jan 23, 2024
9d0a54a
further smaller tweaks
jpkrohling Jan 23, 2024
fe37bed
marked new sections as experimental
jpkrohling Jan 23, 2024
929fe93
fix spec-compliance-matrix
jpkrohling Jan 23, 2024
95ac0a5
Further suggestions from code review
jpkrohling Jan 31, 2024
9cd334c
s/propagator-classes/categories
jpkrohling Jan 31, 2024
d02b751
s/classes/categories/
jpkrohling Jan 31, 2024
1109bd0
change changelog log
jpkrohling Feb 1, 2024
3147dbe
move response propagators to new experimental section
jpkrohling Feb 1, 2024
9733977
revert changes to new env var
jpkrohling Feb 14, 2024
9f997e3
Add image with use-case
jpkrohling Feb 22, 2024
b1ca0b8
fix markdownlint
jpkrohling Feb 22, 2024
cbeb61c
set status as mixed for context
jpkrohling Feb 22, 2024
e325b2f
s/Platforms/Instrumentation Libraries
jpkrohling Feb 28, 2024
9981812
clarify that trace context shouldn't go more than one hop back
jpkrohling Mar 6, 2024
0691c97
add stability markers
jpkrohling Mar 11, 2024
32c43ee
make markdown-toc
jpkrohling Mar 11, 2024
dd838d8
apply suggestion from jmacd
jpkrohling Apr 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ release.

### Context

- Add response propagators.
([#3825](https://github.com/open-telemetry/opentelemetry-specification/pull/3825))

### Traces

### Metrics
Expand Down
Binary file added internal/img/response-propagator-use-case.png
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
1 change: 1 addition & 0 deletions spec-compliance-matrix.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,7 @@ Disclaimer: this list of features is still a work in progress, please refer to t
| Get current Context | | N/A| + | + | + | + | + | + | + | + | + | + |
| Composite Propagator | | + | + | + | + | + | + | + | + | + | + | + |
| Global Propagator | | + | + | + | + | + | + | + | + | + | + | + |
| [ResponsePropagator](specification/context/api-propagators.md#response-propagator)| | - | - | - | - | - | - | - | - | - | - | - |
| TraceContext Propagator | | + | + | + | + | + | + | + | + | + | + | + |
| B3 Propagator | | + | + | + | + | + | + | + | + | + | + | + |
| Jaeger Propagator | | + | + | + | + | + | + | | + | + | - | + |
Expand Down
1 change: 1 addition & 0 deletions specification/configuration/file-configuration.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,7 @@ Interpret [configuration model](#in-memory-configuration-model) and return SDK c
* `MeterProvider`
* `LoggerProvider`
* `Propagators`
* `ResponsePropagators`

The multiple responses MAY be returned using a tuple, or some other data
structure encapsulating the components.
Expand Down
Loading
Loading