Skip to content

Clarify .NET profiler overhead guidance by install method. - #39132

Merged
anandreshmi46 merged 14 commits into
masterfrom
reshmi/dotnet-profiler-reduce-overhead-troubleshooting
Aug 19, 2026
Merged

Clarify .NET profiler overhead guidance by install method.#39132
anandreshmi46 merged 14 commits into
masterfrom
reshmi/dotnet-profiler-reduce-overhead-troubleshooting

Conversation

@anandreshmi46

Copy link
Copy Markdown
Contributor

Add platform tabs under machine-wide profiling so Linux SSI, manual Linux, and Windows each have scoped reduce-overhead steps without mixing IIS-only advice into SSI flows.

What does this PR do? What is the motivation?

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

Add platform tabs under machine-wide profiling so Linux SSI, manual Linux, and Windows each have scoped reduce-overhead steps without mixing IIS-only advice into SSI flows.

Co-authored-by: Cursor <cursoragent@cursor.com>
@anandreshmi46
anandreshmi46 requested a review from a team as a code owner August 12, 2026 12:43
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@chrisnas chrisnas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jhgilbert jhgilbert added the astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg label Aug 12, 2026

@rtrieu rtrieu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

some minor feedback for your consideration

Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md
Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
@anandreshmi46
anandreshmi46 requested a review from rtrieu August 13, 2026 10:17
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
@datadog-datadog-prod-us1-2

This comment has been minimized.

Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
To reduce overhead:

- Use `DD_PROFILING_ENABLED=auto` instead of `true` when enabling profiling host-wide. With `auto`, profiling starts only when the application has run for more than 30 seconds **and** at least one trace was created.
- Use [instrumentation rules][6] to limit which processes receive SSI.

@rtrieu rtrieu Aug 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this link needs to be placed within the tab in order to render

Suggested change
- Use [instrumentation rules][6] to limit which processes receive SSI.
- Use [instrumentation rules][101] to limit which processes receive SSI.

@rtrieu rtrieu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hi @anandreshmi46 noticed a link isn't rendering, and added a few comments. please request a re-review when ready!

Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
- Use `DD_PROFILING_ENABLED=auto` instead of `true` when enabling profiling host-wide. With `auto`, profiling starts only when the application has run for more than 30 seconds **and** at least one trace was created.
- Use [instrumentation rules][6] to limit which processes receive SSI.
- Set `DD_PROFILING_ENABLED=false` in a specific process environment to opt that process out of profiling. Tracing is not affected.

@rtrieu rtrieu Aug 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
[6]: /tracing/trace_collection/single-step-apm/linux/#define-instrumentation-rules

Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
Comment thread hugo/content/en/profiler/profiler_troubleshooting/dotnet.md Outdated
anandreshmi46 and others added 7 commits August 14, 2026 11:15
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
@anandreshmi46
anandreshmi46 requested a review from rtrieu August 17, 2026 09:08
anandreshmi46 and others added 2 commits August 17, 2026 11:13
Move link definitions out of Hugo tabs to the page footer so instrumentation rules and enabling links render correctly.

Co-authored-by: Cursor <cursoragent@cursor.com>

@rtrieu rtrieu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hi @anandreshmi46, the links must be within the tabs in order for them to work.

@@ -147,8 +145,6 @@ If it is set to another value or not set at all, the profiler is disabled.

Note that the following message could appear, but it does not impact Datadog profiling: `Profiler signal handler has been replaced. Restoring it.` This indicates only that the Datadog signal handler is reinstalled when it was overwritten.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
[4]: /profiler/enabling/dotnet/?tab=linux#configuration

@@ -85,8 +85,6 @@ If you've configured the profiler and don't see profiles in the profile search p
- The application ran for more than 30 seconds
If it is set to another value or not set at all, the profiler is disabled.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
[4]: /profiler/enabling/dotnet/?tab=linux#configuration

@@ -147,8 +145,6 @@ If it is set to another value or not set at all, the profiler is disabled.

Note that the following message could appear, but it does not impact Datadog profiling: `Profiler signal handler has been replaced. Restoring it.` This indicates only that the Datadog signal handler is reinstalled when it was overwritten.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No description provided.

@@ -200,8 +196,6 @@ The default profiler log directory is `%ProgramData%\Datadog .NET Tracer\logs\`.

5. Check the other HTTP codes for possible errors such as 403 for invalid API key.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No description provided.

anandreshmi46 and others added 2 commits August 17, 2026 19:50
Reference links inside tabs must be defined within the same tab to render correctly in the docs site.

Co-authored-by: Cursor <cursoragent@cursor.com>
@anandreshmi46
anandreshmi46 merged commit 3952e9e into master Aug 19, 2026
24 checks passed
@anandreshmi46
anandreshmi46 deleted the reshmi/dotnet-profiler-reduce-overhead-troubleshooting branch August 19, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants