From 4e3ee59e976e38f80a5eb507a022f9bcf4114320 Mon Sep 17 00:00:00 2001 From: Tyler Yahn Date: Tue, 6 Apr 2021 10:40:51 -0700 Subject: [PATCH 1/2] Update spec-compliance-matrix Zipkin section for Go --- spec-compliance-matrix.md | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/spec-compliance-matrix.md b/spec-compliance-matrix.md index 3325780e100..0b1f1856c05 100644 --- a/spec-compliance-matrix.md +++ b/spec-compliance-matrix.md @@ -163,19 +163,19 @@ Note: Support for environment variables is optional. | Honors throttling response | X | [-](https://github.com/open-telemetry/opentelemetry-go/issues/1632) | | - | + | + | - | | | | - | - | | Multi-destination spec compliance | X | + | | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1109) | | - | | | | - | - | | [Zipkin](specification/trace/sdk_exporters/zipkin.md) | | | | | | | | | | | | | -| Zipkin V1 JSON | X | | + | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1173) | - | - | - | - | | - | - | -| Zipkin V1 Thrift | X | | + | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1174) | - | - | - | - | | - | - | +| Zipkin V1 JSON | X | - | + | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1173) | - | - | - | - | | - | - | +| Zipkin V1 Thrift | X | - | + | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1174) | - | - | - | - | | - | - | | Zipkin V2 JSON | * | + | + | | + | + | - | + | + | + | + | + | -| Zipkin V2 Protobuf | * | | + | | + | - | + | | - | | - | - | -| Service name mapping | | + | + | + | + | + | + | + | + | | + | + | +| Zipkin V2 Protobuf | * | - | + | | + | - | + | | - | | - | - | +| Service name mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/issues/1777) | + | + | + | + | + | + | + | | + | + | | SpanKind mapping | | + | + | + | + | + | + | + | + | | + | + | -| InstrumentationLibrary mapping | | | + | - | + | + | - | - | + | | + | + | +| InstrumentationLibrary mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | - | + | + | - | - | + | | + | + | | Boolean attributes | | + | + | + | + | + | + | + | + | | + | + | -| Array attributes | | + | + | + | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1110) | + | + | + | + | | + | + | -| Status mapping | | + | + | + | + | + | + | + | + | | + | + | -| Error Status mapping | | | + | | | + | | | | | + | - | +| Array attributes | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | + | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1110) | + | + | + | + | | + | + | +| Status mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | + | + | + | + | + | + | | + | + | +| Error Status mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | | | + | | | | | + | - | | Event attributes mapping to Annotations | | + | + | + | + | + | + | + | + | | + | + | -| Integer microseconds in timestamps | | | + | | + | + | | | | | + | + | +| Integer microseconds in timestamps | | N/A| + | | + | + | | | | | + | + | | [Jaeger](specification/trace/sdk_exporters/jaeger.md) | | | | | | | | | | | | | | Jaeger Thrift over UDP | * | + | | | + | + | | | | | + | + | | Jaeger Protobuf via gRPC | * | - | + | | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1437) | - | | | | | - | - | From 5d019e361920830fc8cbbbfb6712a17f057b00eb Mon Sep 17 00:00:00 2001 From: Tyler Yahn Date: Wed, 7 Apr 2021 13:34:45 -0700 Subject: [PATCH 2/2] Update based up latest changes to otel-go --- spec-compliance-matrix.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/spec-compliance-matrix.md b/spec-compliance-matrix.md index 358551580c6..f04dec7b4e9 100644 --- a/spec-compliance-matrix.md +++ b/spec-compliance-matrix.md @@ -169,11 +169,11 @@ Note: Support for environment variables is optional. | Zipkin V2 Protobuf | * | - | + | | + | - | + | | - | | - | - | | Service name mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/issues/1777) | + | + | + | + | + | + | + | | + | + | | SpanKind mapping | | + | + | + | + | + | + | + | + | | + | + | -| InstrumentationLibrary mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | - | + | + | - | - | + | | + | + | +| InstrumentationLibrary mapping | | + | + | - | + | + | - | - | + | | + | + | | Boolean attributes | | + | + | + | + | + | + | + | + | | + | + | -| Array attributes | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | + | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1110) | + | + | + | + | | + | + | -| Status mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | + | + | + | + | + | + | | + | + | -| Error Status mapping | | [-](https://github.com/open-telemetry/opentelemetry-go/pull/1688) | + | | | + | | | | | + | - | +| Array attributes | | + | + | + | [-](https://github.com/open-telemetry/opentelemetry-python/issues/1110) | + | + | + | + | | + | + | +| Status mapping | | + | + | + | + | + | + | + | + | | + | + | +| Error Status mapping | | + | + | | | + | | | | | + | - | | Event attributes mapping to Annotations | | + | + | + | + | + | + | + | + | | + | + | | Integer microseconds in timestamps | | N/A| + | | + | + | | | | | + | + | | [Jaeger](specification/trace/sdk_exporters/jaeger.md) | | | | | | | | | | | | |