Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distributed Tracing GA Plan #2071

Open
12 tasks
bachuv opened this issue Feb 2, 2022 · 11 comments
Open
12 tasks

Distributed Tracing GA Plan #2071

bachuv opened this issue Feb 2, 2022 · 11 comments
Assignees

Comments

@bachuv
Copy link
Collaborator

bachuv commented Feb 2, 2022

DurableTask

  • Merge OpenTelemetry PR (includes removing the old implementation and adding integration tests) [1wk]
  • Add exception information [3d]
  • (Optional) Add timer information
  • Add docs and samples about how to use it with App Insights, Zipkin, and other telemetry exporters [1d]

Durable Functions

  • Consume information from DurableTask [3d]
  • Add entity information [2d]
  • Remove old implementation [1d]
  • Add integration tests [1d]
  • Add docs and samples about how to use it with App Insights, Zipkin, and other telemetry exporters [1d]

Out of Proc

(dependent on Functions releasing a collector or its equivalent

  • Test with the Functions collector (or equivalent)
  • Add docs and samples about how to use it with App Insights, Zipkin, and other telemetry exporters [1d]

Miscellaneous

  • (Optional) Investigate getting ILogger traces to also correlate (e.g. log.LogInformation()) [1-3d]
@marnilss
Copy link

marnilss commented Jun 7, 2022

Is there any update on this?

I happened to see this page: https://github.com/Azure/azure-functions-durable-extension/blob/dev/samples/correlation-csharp/getting-started.md
Is it related, and how? Is it only for a preview or is it in a released version?

@cgillum
Copy link
Collaborator

cgillum commented Jun 8, 2022

@marnilss the above link refers to a preview, but that preview is available in the current releases. That said, we're redoing the implementation so that it works with all our backend storage providers and so that it's compatible with OpenTelemetry. This issue tracks work required for the rewrite.

@uchitha
Copy link

uchitha commented Jul 12, 2022

@cgillum We're currently reviewing our logging to improve on distributed tracing capabilities. We got multiple durable functions taking part in business transactions that we'd like to track. All of our functions use App Insights. Do you recommend going ahead with the existing implementation at this point or waiting until the new implementation comes out? We are happy to wait if it provides us with more support/predictability. Also aware that the existing implementation lacks support for certain scenarios (like entity funcs)

@cgillum
Copy link
Collaborator

cgillum commented Jul 12, 2022

@uchitha I'd recommend waiting for the new implementation if you can. It will definitely provide you with better support and predictability since we want to deprecate/remove the existing previous functionality.

@marnilss
Copy link

marnilss commented Feb 6, 2024

Apparently it is in preview now !!! https://github.com/Azure/azure-functions-durable-extension/tree/dev/samples/distributed-tracing/v2/DistributedTracingSample

OR is it something else, that looks like what we have been waiting for?

@cgillum
Copy link
Collaborator

cgillum commented Feb 7, 2024

@marnilss yes, it's in preview now (finally)! You can find the official announcement here: https://azure.microsoft.com/en-us/updates/public-preview-distributed-tracing-v2-for-durable-functions/.

Please give it a try and give us feedback! More data will help us get to GA faster. :)

@gha-zund
Copy link

we would really love to try that out, but there are two big must-haves missing for us:

  • durable entity for isolated function
  • send tracing out via OpenTelemetry

we are standing in front a decision:
implement tracing all by our own (which would mean a significant time investment).
or wait until this is available (but ... for how long?)

is there a possibility to push this topic? (user voice or something?)

@lsl-2020
Copy link

lsl-2020 commented Apr 26, 2024

we would really love to try that out, but there are two big must-haves missing for us:

  • durable entity for isolated function
  • send tracing out via OpenTelemetry

we are standing in front a decision: implement tracing all by our own (which would mean a significant time investment). or wait until this is available (but ... for how long?)

is there a possibility to push this topic? (user voice or something?)

@gha-zund For #2 item, they are working on supporting OTel now. You can redirect to the corresponding links starting from here: Azure/azure-functions-host#9273.

@gha-zund
Copy link

thanks for the hint @lsl-2020, I was not aware of that!
Great to read that there is something ongoing! :)

@lsl-2020
Copy link

Hi @cgillum and other experts, since OpenTelemetry support PR is merged in both worker side and host side. Any updates on current thread? Do we have an ETA of when we can adopt DT in Azure Functions other than Application Insights?

@julealgon
Copy link

julealgon commented May 24, 2024

I'm also looking for updates on this. I'm in the process of instrumenting the last applications on our side with OTEL and stumbled upon the problem of configuring it on Azure Functions projects.

I'm a bit confused as to why this was closed:

Is this issue here supposed to be the central place for tracking progress on OpenTelemetry support in Azure Functions?

EDIT:

Apparently OTEL is now supported. Found the documentation here:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants