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

Add EnableDiagnostics sub-configs for IPC, debugger, and profiler #90159

Merged

Conversation

tommcdon
Copy link
Member

@tommcdon tommcdon commented Aug 8, 2023

Today we have one config knob for enabling/disabling all of diagnostics. This PR adds additional granular diagnostic environment variables for the Diagnostics IPC channel, Debugger, and Profiler. This can be useful in production scenarios where fine grain control might be needed to disable creation of the debugger IPC vs. debugger pipes. This came up in a discussion in #86259 (comment).

@tommcdon tommcdon added this to the 8.0.0 milestone Aug 8, 2023
@tommcdon tommcdon requested review from davmason and hoyosjs August 8, 2023 15:24
@ghost ghost assigned tommcdon Aug 8, 2023
@ghost
Copy link

ghost commented Aug 8, 2023

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Issue Details

Today we have one config knob for enabling/disabling all of diagnostics. This PR adds additional granular diagnostic environment variables for the Diagnostics IPC channel, Debugger, and Profiler. This can be useful in production scenarios where fine grain control might be needed to disable creation of the debugger IPC vs. debugger pipes. This came up in a discussion in #86259 (comment).

Author: tommcdon
Assignees: -
Labels:

area-Diagnostics-coreclr

Milestone: 8.0.0

@tommcdon tommcdon requested a review from mikem8361 August 8, 2023 21:30
Copy link
Member

@davmason davmason left a comment

Choose a reason for hiding this comment

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

LGTM

@tommcdon tommcdon merged commit d6d19ad into dotnet:main Aug 8, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants