Azure SDK Tracing Nickel Semester (Jul-Dec21)

[lmolkova]

Azure SDK tracing distributed sample app

• .NET app draft
• Java app draft
• Python app Add python app to distributed tracing sample app azure-sdk-for-python#19971
• JS app Add node.js app to distributed tracing sample app azure-sdk-for-js#16603, PR Add nodejs container and demo lmolkova/azuresdk_tracing_demo#1
• Publish on samples, docs and k8s support

Azure SDK support in .NET OTel

• Add wildcard support to AddLegacySource  open-telemetry/opentelemetry-dotnet#2183
• Add support for producing ActivitySource activities azure-sdk-for-net#16375 - Produce ActivitySource events azure-sdk-for-net#23192

HTTP telemetry duplication:

• Java [BUG] HTTP spans have wrong parent in ApplicationInsights azure-sdk-for-java#22272
• JS
• Python

HTTP instrumentation consistency

HTTP span naming:

HTTP spans have /path?query name and OTel spec says it should be HTTP VERB

• .NET - won't change until conventions are stable (since shipped in stable version already)
• Java - Http spans minor fixes azure-sdk-for-java#24954
• JS - [core] - Use correct span names for tracing policies azure-sdk-for-js#19838
• Python

x-ms-request-id and x-ms-client-request-id attributes consistency

• .NET (requestId, serviceRequestId)
• Java (x-ms-request-id) Http spans minor fixes azure-sdk-for-java#24954
• Python (x-ms-request-id, x-ms-client-request-id)
• JS (requestId, serviceRequestId)

Investiage OTel policy place in HTTP pipeline (should be per try)

• .NET
• Java - per try: Http spans minor fixes azure-sdk-for-java#24954
• JS - [core] - tracingPolicy only invoked once on retries azure-sdk-for-js#18125
• Python

Azure SDK span conventions

• Publish current versioned schemas (including messaging) Add formal Azure SDK tracing conventions #3260
  • write a blog explaining why we have a different one and path going forward
• Add mapping to common schema
  • draft: https://github.com/microsoft/common-schema/pull/262

Misc

• Java: [BUG] Tracing is broken for EventHub binder azure-sdk-for-java#23035
• Change diagnostic-id spelling to comply with other languages azure-sdk-for-java#23141

Cosmos DB (Java): Azure/azure-sdk-for-java#23032

• Allow client spans creation for convenience calls
• Allow making spans current (to work with OTel auto-instrumentation)
• CosmosDb to use those APIs
  • investigate OTel Scope usage in reactor Allow reactor instrumentation to pick up spans from reactor context open-telemetry/opentelemetry-java-instrumentation#4159

Populate tracer information with SDK name and version

To track usage of tracing per SDK, we can leverage Azure Monitor pipelines, if we populate tracer info (currently Azure-OpenTelemetry, no version).

• .NET
• Java
• Python
• JS - [core] - Extract OpenTelemetry to a separate package azure-sdk-for-js#19734

[github-actions]

Hi @lmolkova, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.