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

Remove Included Propagators section from trace API spec #1556

Merged
merged 6 commits into from
Mar 22, 2021
Merged

Remove Included Propagators section from trace API spec #1556

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
50cb211
Remove Included Propagators section from trace API spec
MrAlias Mar 17, 2021
959e568
Add changes to changelog
MrAlias Mar 17, 2021
c81da7d
Remove section from table of contents
MrAlias Mar 17, 2021
d33cba4
Merge branch 'main' into fix-1544
MrAlias Mar 19, 2021
1558bfa
Leave breadcrumbs
MrAlias Mar 20, 2021
598986d
Merge branch 'main' into fix-1544
carlosalberto Mar 22, 2021
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
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ release.
- Clarify the description for the `TraceIdRatioBased` sampler needs to include the sampler's sampling ratio. ([#1536](/~https://github.com/open-telemetry/opentelemetry-specification/pull/1536))
- Define the fallback tracer name for invalid values.
([#1534](/~https://github.com/open-telemetry/opentelemetry-specification/pull/1534))
- Remove the Included Propagators section from trace API specification that was a duplicate of the Propagators Distribution of the context specification. ([#1556](/~https://github.com/open-telemetry/opentelemetry-specification/pull/1556))

### Metrics

Expand Down
10 changes: 0 additions & 10 deletions specification/trace/api.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ Table of Contents
* [Wrapping a SpanContext in a Span](#wrapping-a-spancontext-in-a-span)
* [SpanKind](#spankind)
* [Concurrency](#concurrency)
* [Included Propagators](#included-propagators)

</details>

Expand Down Expand Up @@ -748,15 +747,6 @@ be called concurrently.

**Link** - Links are immutable and safe to be used concurrently.

## Included Propagators

The API layer or an extension package MUST include the following `Propagator`s:

* A `TextMapPropagator` implementing the [W3C TraceContext Specification](https://www.w3.org/TR/trace-context/).

See [Propagators Distribution](../context/api-propagators.md#propagators-distribution)
for how propagators are to be distributed.

MrAlias marked this conversation as resolved.
Show resolved Hide resolved
## Behavior of the API in the absence of an installed SDK

In general, in the absence of an installed SDK, the Trace API is a "no-op" API.
Expand Down