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: Adds configuration to disable network level tracing in sdk permanently #4333

Conversation

sourabh1007
Copy link
Contributor

@sourabh1007 sourabh1007 commented Feb 29, 2024

Description

It was decided that we are going to stop emitting network level traces to avoid confusion in customer.
Because it has information related to RNTBD calls which is an internal protocol, without much information documented publicly.

  1. Updated Sample Project also: to get it reflected here https://learn.microsoft.com/en-us/azure/cosmos-db/nosql/sdk-observability?tabs=dotnet
  2. Removed all the logic related to network level traces and corresponding tests.

Type of change

  • [] This change requires a documentation update

kirankumarkolli
kirankumarkolli previously approved these changes Mar 4, 2024
@sourabh1007 sourabh1007 added the auto-merge Enables automation to merge PRs label Mar 5, 2024
Microsoft.Azure.Cosmos/src/DocumentClient.cs Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enables automation to merge PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants