From fbea4af0badd9ee6c534ab78fedee91bcd6c4a37 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 22 Nov 2021 22:15:19 -0800 Subject: [PATCH 01/12] Mark Metrics SDK as Stable --- specification/metrics/sdk.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/specification/metrics/sdk.md b/specification/metrics/sdk.md index fc2b114c70a..ad7060bd310 100644 --- a/specification/metrics/sdk.md +++ b/specification/metrics/sdk.md @@ -1,6 +1,6 @@ # Metrics SDK -**Status**: [Feature-freeze](../document-status.md) +**Status**: [Stable](../document-status.md)
Table of Contents From 4bfca4d63a0ecfe7abad9cc70e3b319d5d529079 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 22 Nov 2021 22:20:48 -0800 Subject: [PATCH 02/12] update changelog --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index b60c6d8b75a..a9ca2af29d5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -41,6 +41,8 @@ release. [#2032](https://github.com/open-telemetry/opentelemetry-specification/pull/2061) - Changed default Prometheus Exporter host from `0.0.0.0` to `localhost`. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) +- Mark Metrics SDK spec as Stable. + ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2150)) ### Logs From 3be1b82a72c084c91234dc3948b2d50e34288c5b Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 29 Nov 2021 09:45:29 -0800 Subject: [PATCH 03/12] change the doc status to Mixed --- specification/metrics/sdk.md | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/specification/metrics/sdk.md b/specification/metrics/sdk.md index ad7060bd310..d695dd974f9 100644 --- a/specification/metrics/sdk.md +++ b/specification/metrics/sdk.md @@ -1,6 +1,6 @@ # Metrics SDK -**Status**: [Stable](../document-status.md) +**Status**: [Mixed](../document-status.md)
Table of Contents @@ -47,6 +47,8 @@ ## MeterProvider +**Status**: [Stable](../document-status.md) + A `MeterProvider` MUST provide a way to allow a [Resource](../resource/sdk.md) to be specified. If a `Resource` is specified, it SHOULD be associated with all the metrics produced by any `Meter` from the `MeterProvider`. The [tracing SDK @@ -416,6 +418,8 @@ instruments that record negative measurements, e.g. `UpDownCounter` or `Observab ## Attribute limits +**Status**: [Stable](../document-status.md) + Attributes which belong to Metrics are exempt from the [common rules of attribute limits](../common/common.md#attribute-limits) at this time. Attribute truncation or deletion could affect identity of metric time @@ -423,6 +427,8 @@ series and the topic requires further analysis. ## Exemplar +**Status**: [Experimental](../document-status.md) + Exemplars are example data points for aggregated data. They provide specific context to otherwise general aggregations. Exemplars allow correlation between aggregated metric data and the original API calls where measurements are @@ -587,6 +593,8 @@ measurements using the equivalent of the following naive algorithm: ## MetricReader +**Status**: [Stable](../document-status.md) + `MetricReader` is an interface which provides the following capabilities: * Collecting metrics from the SDK. @@ -709,6 +717,8 @@ from `MetricReader` and start a background task which calls the inherited ## MetricExporter +**Status**: [Stable](../document-status.md) + `MetricExporter` defines the interface that protocol-specific exporters MUST implement so that they can be plugged into OpenTelemetry SDK and support sending of telemetry data. @@ -930,6 +940,8 @@ errors/exceptions are taken care of. ## Compatibility requirements +**Status**: [Stable](../document-status.md) + All the metrics components SHOULD allow new methods to be added to existing components without introducing breaking changes. @@ -938,6 +950,8 @@ existing methods without introducing breaking changes, if possible. ## Concurrency requirements +**Status**: [Stable](../document-status.md) + For languages which support concurrent execution the Metrics SDKs provide specific guarantees and safeties. From 0250ba4570e2cb5089ae3829f551ce536ef1de18 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 29 Nov 2021 09:48:28 -0800 Subject: [PATCH 04/12] update changelog --- CHANGELOG.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index a9ca2af29d5..a5ee9684c09 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -28,6 +28,7 @@ release. ([#2124](https://github.com/open-telemetry/opentelemetry-specification/pull/2124)) - Remove the concept of supported temporality, keep preferred. ([#2154](https://github.com/open-telemetry/opentelemetry-specification/pull/2154)) +<<<<<<< HEAD - Mark In-memory, OTLP and Stdout exporter specs as Stable. ([#2175](https://github.com/open-telemetry/opentelemetry-specification/pull/2175)) - Add to the supplemental guidelines for metric SDK authors text about implementing @@ -42,6 +43,10 @@ release. - Changed default Prometheus Exporter host from `0.0.0.0` to `localhost`. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) - Mark Metrics SDK spec as Stable. +======= +- Mark Metrics SDK spec as Mixed, with `MeterProvider`, `MetricReader` and + `MetricExporter` marked as Stable. +>>>>>>> 68952d1 (update changelog) ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2150)) ### Logs From a753771e2d1215e1b9b7950043bcb9c51bb83e66 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 29 Nov 2021 10:28:06 -0800 Subject: [PATCH 05/12] mark exemplar and baggage as experimental --- specification/metrics/sdk.md | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/specification/metrics/sdk.md b/specification/metrics/sdk.md index d695dd974f9..e0e64fa977b 100644 --- a/specification/metrics/sdk.md +++ b/specification/metrics/sdk.md @@ -182,9 +182,10 @@ are the inputs: outputs metric points that use aggregation temporality (e.g. Histogram, Sum), the SDK SHOULD handle the aggregation temporality based on the temporality of each [MetricReader](#metricreader) instance. - * The `exemplar_reservoir` (optional) to use for storing exemplars. - This should be a factory or callback similar to aggregation which allows - different reservoirs to be chosen by the aggregation. + * **Status**: [Experimental](../document-status.md) - the `exemplar_reservoir` + (optional) to use for storing exemplars. This should be a factory or + callback similar to aggregation which allows different reservoirs to be + chosen by the aggregation. In order to avoid conflicts, views which specify a name SHOULD have an instrument selector that selects at most one instrument. For the registration From d3ea33c15cd23bf1ec851d5b4658c90b80d8cddd Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Tue, 30 Nov 2021 09:01:06 -0800 Subject: [PATCH 06/12] update the status based on discussion in the spec SIG --- specification/metrics/sdk.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/specification/metrics/sdk.md b/specification/metrics/sdk.md index e0e64fa977b..af80744445a 100644 --- a/specification/metrics/sdk.md +++ b/specification/metrics/sdk.md @@ -182,7 +182,7 @@ are the inputs: outputs metric points that use aggregation temporality (e.g. Histogram, Sum), the SDK SHOULD handle the aggregation temporality based on the temporality of each [MetricReader](#metricreader) instance. - * **Status**: [Experimental](../document-status.md) - the `exemplar_reservoir` + * **Status**: [Feature-freeze](../document-status.md) - the `exemplar_reservoir` (optional) to use for storing exemplars. This should be a factory or callback similar to aggregation which allows different reservoirs to be chosen by the aggregation. @@ -428,7 +428,7 @@ series and the topic requires further analysis. ## Exemplar -**Status**: [Experimental](../document-status.md) +**Status**: [Feature-freeze](../document-status.md) Exemplars are example data points for aggregated data. They provide specific context to otherwise general aggregations. Exemplars allow correlation between From 2aa0ccbc4deadaa4764db6b2c0d59296e02cb7df Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Tue, 30 Nov 2021 11:20:12 -0800 Subject: [PATCH 07/12] rewrap --- specification/metrics/sdk.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/specification/metrics/sdk.md b/specification/metrics/sdk.md index af80744445a..dba2d548348 100644 --- a/specification/metrics/sdk.md +++ b/specification/metrics/sdk.md @@ -182,10 +182,10 @@ are the inputs: outputs metric points that use aggregation temporality (e.g. Histogram, Sum), the SDK SHOULD handle the aggregation temporality based on the temporality of each [MetricReader](#metricreader) instance. - * **Status**: [Feature-freeze](../document-status.md) - the `exemplar_reservoir` - (optional) to use for storing exemplars. This should be a factory or - callback similar to aggregation which allows different reservoirs to be - chosen by the aggregation. + * **Status**: [Feature-freeze](../document-status.md) - the + `exemplar_reservoir` (optional) to use for storing exemplars. This should be + a factory or callback similar to aggregation which allows different + reservoirs to be chosen by the aggregation. In order to avoid conflicts, views which specify a name SHOULD have an instrument selector that selects at most one instrument. For the registration From ebad87897f3500dec6a7e72fd52bec06bf01c6b6 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 31 Jan 2022 08:04:18 -0800 Subject: [PATCH 08/12] update changelog --- CHANGELOG.md | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index a5ee9684c09..49e8c39dd20 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -28,7 +28,6 @@ release. ([#2124](https://github.com/open-telemetry/opentelemetry-specification/pull/2124)) - Remove the concept of supported temporality, keep preferred. ([#2154](https://github.com/open-telemetry/opentelemetry-specification/pull/2154)) -<<<<<<< HEAD - Mark In-memory, OTLP and Stdout exporter specs as Stable. ([#2175](https://github.com/open-telemetry/opentelemetry-specification/pull/2175)) - Add to the supplemental guidelines for metric SDK authors text about implementing @@ -42,11 +41,11 @@ release. [#2032](https://github.com/open-telemetry/opentelemetry-specification/pull/2061) - Changed default Prometheus Exporter host from `0.0.0.0` to `localhost`. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) -- Mark Metrics SDK spec as Stable. -======= -- Mark Metrics SDK spec as Mixed, with `MeterProvider`, `MetricReader` and - `MetricExporter` marked as Stable. ->>>>>>> 68952d1 (update changelog) +- Mark In-memory, OTLP and Stdout exporter specs as Stable. + ([#2175](https://github.com/open-telemetry/opentelemetry-specification/pull/2175)) +- Mark Metrics SDK spec as Mixed, with most compnents remaining in + Feature-freeze while Attribute Limits, Compatibility Requirements, and + Concurrency Requirements moving to Stable. ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2150)) ### Logs From 09f7a39d811d11c12858b52d057ef461a3057cdb Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 31 Jan 2022 08:07:45 -0800 Subject: [PATCH 09/12] update changelog --- CHANGELOG.md | 2 -- 1 file changed, 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 49e8c39dd20..93f30683915 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -41,8 +41,6 @@ release. [#2032](https://github.com/open-telemetry/opentelemetry-specification/pull/2061) - Changed default Prometheus Exporter host from `0.0.0.0` to `localhost`. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) -- Mark In-memory, OTLP and Stdout exporter specs as Stable. - ([#2175](https://github.com/open-telemetry/opentelemetry-specification/pull/2175)) - Mark Metrics SDK spec as Mixed, with most compnents remaining in Feature-freeze while Attribute Limits, Compatibility Requirements, and Concurrency Requirements moving to Stable. From f7d191bc2ce4c3eff972588d5662f9fc61dfafa2 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 31 Jan 2022 08:11:48 -0800 Subject: [PATCH 10/12] changelog --- CHANGELOG.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 93f30683915..1276a3f2d7a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -41,9 +41,8 @@ release. [#2032](https://github.com/open-telemetry/opentelemetry-specification/pull/2061) - Changed default Prometheus Exporter host from `0.0.0.0` to `localhost`. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) -- Mark Metrics SDK spec as Mixed, with most compnents remaining in - Feature-freeze while Attribute Limits, Compatibility Requirements, and - Concurrency Requirements moving to Stable. +- Mark Metrics SDK spec as Mixed, with most components moving to Stable, while + Exemplar remaining Feature-freeze. ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2150)) ### Logs From 1b230852e4784ed264ddc9db85793f09b134b5d9 Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 31 Jan 2022 08:16:22 -0800 Subject: [PATCH 11/12] update changelog --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 1276a3f2d7a..802f85d3f28 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -43,7 +43,7 @@ release. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) - Mark Metrics SDK spec as Mixed, with most components moving to Stable, while Exemplar remaining Feature-freeze. - ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2150)) + ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2304)) ### Logs From ed35b3dffe44a115a41bf6d1921b16bdcdc94c6b Mon Sep 17 00:00:00 2001 From: Reiley Yang Date: Mon, 31 Jan 2022 08:16:49 -0800 Subject: [PATCH 12/12] update changelog --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 802f85d3f28..7f4f9f11082 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -43,7 +43,7 @@ release. ([#2282](https://github.com/open-telemetry/opentelemetry-specification/pull/2282)) - Mark Metrics SDK spec as Mixed, with most components moving to Stable, while Exemplar remaining Feature-freeze. - ([#2150](https://github.com/open-telemetry/opentelemetry-specification/pull/2304)) + ([#2304](https://github.com/open-telemetry/opentelemetry-specification/pull/2304)) ### Logs