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

profiler: add enable flag to control profiler activation #2840

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Commits on Aug 31, 2024

  1. profiler: add enable flag to control profiler activation

    Add a new 'enable' field to the profiler config, controlled by the
    DD_PROFILING_ENABLED environment variable. This allows users to
    disable profiling even when the Start() function is called.
    
    The enable flag defaults to true, maintaining backwards compatibility.
    When set to false, the profiler will not start, providing a simple way
    to toggle profiling without code changes.
    
    Update tests to cover the new functionality and add logging for the
    new configuration option.
    korECM committed Aug 31, 2024
    Configuration menu
    Copy the full SHA
    df7801c View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2024

  1. Set profiling enablement to true if "auto" is detected

    Updated profiler options to automatically enable profiling if the environment variable "DD_PROFILING_ENABLED" is set to "auto". This change delegates the decision to the Datadog admission controller when "auto" is specified.
    korECM committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    ab2acd9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    3aac285 View commit details
    Browse the repository at this point in the history
  3. Add test for profiler disabled condition

    Implemented a new test to verify that no profiles are received when the profiler is disabled. This helps ensure the profiler respects the DD_PROFILING_ENABLED environment variable.
    korECM committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    f71e851 View commit details
    Browse the repository at this point in the history