Skip to content

Update DDOT diagram to clarify potential confusion with mixed instrumentation #30105

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

Merged
merged 1 commit into from
Jun 25, 2025

Conversation

brett0000FF
Copy link
Contributor

@brett0000FF brett0000FF commented Jun 24, 2025

What does this PR do? What is the motivation?

  • Current diagram could be interpreted to show Datadog tracer and OTel SDK in a single application.
  • Update diagram to make it explicit that you instrument using the DD tracer or the OTel SDK.
  • Expand Host box to show that the application code runs on the same host.

image

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:

Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.

Additional notes

@brett0000FF brett0000FF requested a review from a team as a code owner June 24, 2025 21:22
@github-actions github-actions bot added the Images Images are added/removed with this PR label Jun 24, 2025
Copy link
Contributor

github-actions bot commented Jun 24, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

@brett0000FF brett0000FF changed the title Update DDOT diagram to clarify potential confusion Update DDOT diagram to clarify potential confusion with mixed instrumentation Jun 24, 2025
Copy link
Contributor

Copy link
Contributor

@maycmlee maycmlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@brett0000FF brett0000FF merged commit 8b4d80b into master Jun 25, 2025
19 of 20 checks passed
@brett0000FF brett0000FF deleted the brett.blue/otel-diagram branch June 25, 2025 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Images Images are added/removed with this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants