-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Improvements to APM -> Compatibility -> .NET Core #8960
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love this.
Sorry for the inactivity on this, I was still working through some feedback from the engineering team. Until I finalize that, I will keep this as a draft and re-request approvals if things change. |
…al details to external repositories that have more context around the issues
26ee27d
to
f3cf336
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
What does this PR do?
Adds a new section in the .NET Core compatibility docs to highlight known runtime issues that affect Datadog automatic instrumentation.
Motivation
We discovered another issue that with an older version of .NET Core 2.0
Preview
https://docs-staging.datadoghq.com/zach.montoya/net-core-2.0.0/tracing/compatibility_requirements/dotnet-core/
Additional Notes
N/A
Reviewer checklist