Skip to content

feat: track explicit debugger and profiler configuration - #6202

Open
nargokul wants to merge 1 commit into
aws:master-v2from
nargokul:feat/debugger-profiler-explicit-telemetry
Open

feat: track explicit debugger and profiler configuration#6202
nargokul wants to merge 1 commit into
aws:master-v2from
nargokul:feat/debugger-profiler-explicit-telemetry

Conversation

@nargokul

Copy link
Copy Markdown
Contributor

Issue #, if available:

N/A

Description of changes:

  • Record whether debugger_hook_config and profiler_config were provided directly to an estimator before SDK defaults and training preparation transform those values.
  • Extend the telemetry emitter with opt-in instance attribute fields.
  • Emit x-debuggerHookConfigExplicitlyProvided and x-profilerConfigExplicitlyProvided on estimator.fit.

Testing done:

  • PYTHONPATH=/tmp/sagemaker-v2-test-deps:src python -m pytest -q tests/unit/test_estimator.py (276 passed)
  • PYTHONPATH=/tmp/sagemaker-v2-test-deps:src python -m pytest -q tests/unit/sagemaker/telemetry/test_telemetry_logging.py (19 passed)
  • Black, Flake8, and git diff --check

Merge Checklist

General

  • I certify that the changes I am introducing will be backward compatible.
  • I used the commit message format described in CONTRIBUTING.
  • No S3 or STS clients are introduced by this change.
  • No documentation updates are required for internal telemetry fields.

Tests

  • I have added tests that prove the feature works.
  • I have added unit tests to ensure backward compatibility.
  • The tests are not configured for a specific account.
  • No dependencies are added.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@nargokul
nargokul requested a review from a team as a code owner August 21, 2026 22:20
@nargokul
nargokul deployed to auto-approve August 21, 2026 22:20 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant