Skip to content

Commit 2945f8c

Browse files
Merge branch 'main' into renovate/github-actions-deps
2 parents 7324d57 + 9a46e5b commit 2945f8c

File tree

13 files changed

+51
-28
lines changed

13 files changed

+51
-28
lines changed

CONTRIBUTING.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,8 @@ To check for typos, run the following command:
158158
make misspell
159159
```
160160

161-
> **NOTE**: The `misspell` make target will also fetch and build the tool if
161+
> [!NOTE]
162+
> The `misspell` make target will also fetch and build the tool if
162163
> necessary. You'll need [Go](https://go.dev) to build the spellchecker.
163164
164165
To quickly fix typos, use

oteps/0156-columnar-encoding.md

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,8 @@ the compression ratio. This is illustrated in the following diagram.
190190

191191
![Traffic reduction](img/0156_traffic_reduction_use_case.png)
192192

193-
> Note 1: A fallback mechanism can be used to handle the case where the new protocol is not supported by the target.
193+
> [!NOTE]
194+
> **1:** A fallback mechanism can be used to handle the case where the new protocol is not supported by the target.
194195
> More on this mechanism in this [section](#protocol-extension-and-fallback-mechanism).
195196
196197
#### Phase 2
@@ -274,6 +275,7 @@ service ArrowMetricsService {
274275
}
275276
```
276277

278+
> [!INFORMATION]
277279
> **Unary RPC vs Stream RPC**: We use a stream-oriented protocol **to get rid of the overhead of specifying the schema
278280
> and dictionaries for each batch.** A state will be maintained receiver side to keep track of the schemas and
279281
> dictionaries. The [Arrow IPC format](#arrow-ipc-format) has been designed to follow this pattern and also allows the
@@ -391,7 +393,8 @@ By storing Arrow buffers in a protobuf field of type 'bytes' we can leverage the
391393
protobuf implementations (e.g. C++, Java, Rust) in order to get the most out of Arrow (relying on zero-copy ser/deser
392394
framework).
393395

394-
> Note: By default, ZSTD compression is enabled at the Arrow IPC level in order to benefit from the best compression
396+
> [!NOTE]
397+
> By default, ZSTD compression is enabled at the Arrow IPC level in order to benefit from the best compression
395398
> ratio regardless of the collector configuration. However, this compression can be disabled to enable it at the global
396399
> gRPC level if it makes more sense for a particular configuration.
397400
@@ -450,7 +453,8 @@ To indicate not-retryable errors the server is recommended to use code INVALID_A
450453
details
451454
via `error_message`.
452455

453-
> Note: [Appendix A](#appendix-a---protocol-buffer-definitions) contains the full protobuf definition.
456+
> [!NOTE]
457+
> [Appendix A](#appendix-a---protocol-buffer-definitions) contains the full protobuf definition.
454458
455459
### Mapping OTel Entities to Arrow Records
456460

@@ -489,7 +493,8 @@ same schema are grouped in a homogeneous stream. The first message sent contains
489493
the schema definition and the dictionaries. The following messages will not need to define the schema anymore.
490494
The dictionaries will only be sent again when their content change. The following diagram illustrates this process.
491495

492-
> Note: The approach of using a single Arrow record per OTel entity, which employs list, struct, and union Arrow data
496+
> [!NOTE]
497+
> The approach of using a single Arrow record per OTel entity, which employs list, struct, and union Arrow data
493498
> types, was not adopted mainly due to the inability to sort each level of the OTel hierarchy independently. The mapping
494499
> delineated in this document, on average, provides a superior compression ratio.
495500
@@ -533,7 +538,8 @@ engines.
533538
- The avoidance of complex Arrow data types (like union, list of struct) optimizes compatibility with the Arrow
534539
ecosystem.
535540

536-
> Note: Complex attribute values could also be encoded in protobuf once the `pdata` library provides support for it.
541+
> [!NOTE]
542+
> Complex attribute values could also be encoded in protobuf once the `pdata` library provides support for it.
537543
538544
#### Spans Arrow Mapping
539545

@@ -581,7 +587,8 @@ As usual, each of these Arrow records is sorted by specific columns to optimize
581587
batch of metrics containing a large number of data points sharing the same attributes and timestamp will be highly
582588
compressible (multivariate time-series scenario).
583589

584-
> Note: every OTLP timestamps are represented as Arrow timestamps as Epoch timestamps with nanosecond precision. This representation will
590+
> [!NOTE]
591+
> every OTLP timestamps are represented as Arrow timestamps as Epoch timestamps with nanosecond precision. This representation will
585592
> simplify the integration with the rest of the Arrow ecosystem (numerous time/date functions are supported in
586593
> DataFusion for example).
587594
> Note: aggregation_temporality is represented as an Arrow dictionary with a dictionary index of type int8. This OTLP

oteps/0243-app-telemetry-schema-vision-roadmap.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,10 @@ and [Honeycomb Query Assistant](https://www.honeycomb.io/blog/introducing-query-
109109
are likely to become increasingly prevalent and efficient in the near future,
110110
thanks to the adoption of a schema-first approach.
111111

112-
> **Note: The names and formats of these concepts are still under discussion. A
112+
> [!NOTE]
113+
> The names and formats of these concepts are still under discussion. A
113114
> detailed analysis of pros and cons will be covered later in the document. The
114-
> final decision will be deferred to future dedicated OTEPs.**
115+
> final decision will be deferred to future dedicated OTEPs.
115116
116117
Another problem this proposal aims to address is the inherent complexity of the
117118
ecosystem where OpenTelemetry is utilized but not fully addressed by existing

oteps/0258-env-context-baggage-carriers.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,8 @@ and produced by an arbitrary TextMapPropagator.
6565

6666
Consider the following diagram in the context of process forking:
6767

68-
> Note: The diagram is simply an example and simplification of process forking.
68+
> [!NOTE]
69+
> The diagram is simply an example and simplification of process forking.
6970
> There are other ways to spawn processes which are more performant like
7071
> exec().
7172
@@ -78,7 +79,8 @@ propagate context to the child process such that it can be tied to the parent.
7879
Without `TRACEPARENT`, a tracing backend would not be able to connect the child
7980
process spans to the parent span, forming an end-to-end trace.
8081

81-
> Note: While the below exclusively follows the W3C Specification translated
82+
> [!NOTE]
83+
> While the below exclusively follows the W3C Specification translated
8284
> into environment variables, this proposal is not exclusive to W3C and is
8385
> instead focused on the mechanism of Text Map Propagation with a potential set
8486
> of well-known environment variable names. See the [Core Specification
@@ -102,7 +104,8 @@ This could be set in the environment as follows:
102104
export TRACEPARENT=00-4bf92f3577b34da6a3ce929d0e0e4736-00f067aa0ba902b7-01
103105
```
104106

105-
> Note: The value of TRACEPARENT is a combination of the above field values as
107+
> [!NOTE]
108+
> The value of TRACEPARENT is a combination of the above field values as
106109
> unsigned integer values serialized as ASCII strings, delimited by `-`.
107110
108111
The `tracestate` (lowercase) header is defined in [W3C

oteps/trace/0174-http-semantic-conventions.md

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@ stable state is a crucial step for users and instrumentation authors, as it
1919
allows them to rely on [stability guarantees](../../specification/versioning-and-stability.md#semantic-conventions-stability),
2020
and thus to ship and use stable instrumentation.
2121

22-
> NOTE. This OTEP captures a scope for changes should be done to existing
22+
> [!NOTE]
23+
> This OTEP captures a scope for changes should be done to existing
2324
experimental semantic conventions for HTTP, but does not propose solutions.
2425

2526
## Roadmap for v1.0
@@ -43,7 +44,8 @@ some steps can be worked on in parallel.
4344

4445
## Scope for v1.0: scenarios and open questions
4546

46-
> NOTE. The scope defined here is subject for discussions and can be adjusted
47+
> [!NOTE]
48+
> The scope defined here is subject for discussions and can be adjusted
4749
until this OTEP is merged.
4850

4951
Scenarios and open questions mentioned below must be addressed via separate PRs.
@@ -111,7 +113,8 @@ Some attributes can contain potentially sensitive information. Most likely, by
111113
default web frameworks/http clients should not expose that. For example,
112114
`http.target` has a query string that may contain credentials.
113115

114-
> NOTE. We didn’t omit security concerns from v1.0 on purpose, it’s just not
116+
> [!NOTE]
117+
> We didn’t omit security concerns from v1.0 on purpose, it’s just not
115118
something we’ve fleshed out so far.
116119

117120
### Sampling for noop case
@@ -135,7 +138,8 @@ Need some real-life/expertize here.
135138

136139
### Request/Response body capturing
137140

138-
> NOTE. This is technically out-of-scope, but we should have an idea how to let
141+
> [!NOTE]
142+
> This is technically out-of-scope, but we should have an idea how to let
139143
users do it
140144

141145
There is a lot of user feedback that they want it, but

specification/configuration/data-model.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ weight: 2
1010
<!-- toc -->
1111

1212
- [Overview](#overview)
13-
* [Stability definition](#stability-definition)
13+
* [Versioning policy and stability guarantees](#versioning-policy-and-stability-guarantees)
1414
* [File-based configuration model](#file-based-configuration-model)
1515
+ [YAML file format](#yaml-file-format)
1616
+ [Environment variable substitution](#environment-variable-substitution)
@@ -34,9 +34,9 @@ The data model itself is an abstraction with multiple built-in representations:
3434
* [File-based configuration model](#file-based-configuration-model)
3535
* [SDK in-memory configuration model](./sdk.md#in-memory-configuration-model)
3636

37-
### Stability definition
37+
### Versioning policy and stability guarantees
3838

39-
See `opentelemetry-configuration` [stability definition](https://github.com/open-telemetry/opentelemetry-configuration?tab=readme-ov-file#stability-definition).
39+
See `opentelemetry-configuration` [versioning policy](https://github.com/open-telemetry/opentelemetry-configuration/blob/main/VERSIONING.md).
4040

4141
### File-based configuration model
4242

specification/context/env-carriers.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,8 @@ Windows.
159159

160160
## Supplementary Guidelines
161161

162-
> **IMPORTANT**: This section is non-normative and provides implementation
162+
> [!IMPORTANT]
163+
> This section is non-normative and provides implementation
163164
> guidance only. It does not add requirements to the specification.
164165
165166
Language implementations of OpenTelemetry have flexibility in how they implement

specification/entities/data-model.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ the `service` entity is related to the `process` entity. The process normally
3939
also runs on a host, so we say that the `process` entity is related to the
4040
`host` entity.
4141

42-
> Note: Entity relationship modelling will be refined in future specification
42+
> [!NOTE]
43+
> Entity relationship modelling will be refined in future specification
4344
> work.
4445
4546
The data model below defines a logical model for an entity (irrespective of the
@@ -69,7 +70,8 @@ the identity was generated from the process itself, e.g. via SDK, or by an
6970
OpenTelemetry Collector running on the same host, or by some other system
7071
describing the process.
7172

72-
> Aside: There are many ways to accomplish repeatable identifying attributes
73+
> [!TIP]
74+
> There are many ways to accomplish repeatable identifying attributes
7375
> across multiple observers. While many successful systems rely on pushing down
7476
> identity from a central registry or knowledge store, OpenTelemetry must
7577
> support all possible scenarios.

specification/metrics/data-model.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,8 @@ for telemetry collection purposes. The best example of this is the Histogram
224224
metric where raw events are recorded in a compressed format rather than
225225
individual timeseries.
226226

227-
> Note: The above picture shows how one instrument can transform events into
227+
> [!NOTE]
228+
> The above picture shows how one instrument can transform events into
228229
> more than one type of metric stream. There are caveats and nuances for when
229230
> and how to do this. Instrument and metric configuration are outlined
230231
> in the [metrics API specification](api.md).
@@ -1238,7 +1239,8 @@ sums to be reported, the timeseries model we target does not support delta
12381239
counters. To this end, converting from delta to cumulative needs to be defined
12391240
so that backends can use this mechanism.
12401241

1241-
> Note: This is not the only possible Delta to Cumulative algorithm. It is
1242+
> [!NOTE]
1243+
> This is not the only possible Delta to Cumulative algorithm. It is
12421244
> just one possible implementation that fits the OTel Data Model.
12431245
12441246
Converting from delta points to cumulative point is inherently a stateful

specification/resource/README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ through their `kubectl` or Kubernetes UIs to find the specific process
6262
generating telemetry. This is as important as being able to uniquely identify
6363
one process from another.
6464

65-
> Aside: Observability signals SHOULD be actionable. Knowing a process is
65+
> [!TIP]
66+
> Observability signals SHOULD be actionable. Knowing a process is
6667
> struggling is not as useful as being able to scale up a deployment to take
6768
> load off the struggling process.
6869

0 commit comments

Comments
 (0)