From a5e17ba900e97706cd83ec37830bcc3739aa10d2 Mon Sep 17 00:00:00 2001 From: Matthew Wear Date: Wed, 26 Aug 2020 14:50:19 -0700 Subject: [PATCH 1/3] specify that resources map to jaeger process tags --- specification/trace/sdk_exporters/jaeger.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/specification/trace/sdk_exporters/jaeger.md b/specification/trace/sdk_exporters/jaeger.md index e96f3a32534..adaff732593 100644 --- a/specification/trace/sdk_exporters/jaeger.md +++ b/specification/trace/sdk_exporters/jaeger.md @@ -16,6 +16,12 @@ TBD This section discusses the details of the transformations between OpenTelemetry and Jaeger. +### Resource + +OpenTelemetry resources MUST be mapped to Jaeger process tags. Multiple resources can exist for a +single process and exporters need to account for this by grouping spans by resource, and exporting +them in batches. + ### InstrumentationLibrary OpenTelemetry Span's `InstrumentationLibrary` MUST be reported as span `tags` to Jaeger using the following mapping. From 56fb9c5801f294e3a1cf08d034f59b6485d6617b Mon Sep 17 00:00:00 2001 From: Matthew Wear Date: Wed, 26 Aug 2020 15:49:37 -0700 Subject: [PATCH 2/3] add changelog entry --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 2ce2f2ac592..fbb42e4ee6b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -9,6 +9,8 @@ release. New: +- Define resource mapping for Jaeger exporters + ([#891](https://github.com/open-telemetry/opentelemetry-specification/pull/891)) - Add resource semantic conventions for operating systems ([#693](https://github.com/open-telemetry/opentelemetry-specification/pull/693)) - Clarification of the behavior of the Trace API, re: context propagation, in From 85fc7d75cb55ee677b1b3ffa29efe49a0f7a7556 Mon Sep 17 00:00:00 2001 From: Matthew Wear Date: Thu, 27 Aug 2020 11:29:37 -0700 Subject: [PATCH 3/3] be less prescriptive on handling multiple resources --- specification/trace/sdk_exporters/jaeger.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/specification/trace/sdk_exporters/jaeger.md b/specification/trace/sdk_exporters/jaeger.md index adaff732593..d5159730278 100644 --- a/specification/trace/sdk_exporters/jaeger.md +++ b/specification/trace/sdk_exporters/jaeger.md @@ -19,8 +19,7 @@ and Jaeger. ### Resource OpenTelemetry resources MUST be mapped to Jaeger process tags. Multiple resources can exist for a -single process and exporters need to account for this by grouping spans by resource, and exporting -them in batches. +single process and exporters need to handle this case accordingly. ### InstrumentationLibrary