Skip to content

feat: add DGD overrides in profiler - #6546

Merged
tedzhouhk merged 5 commits into
hzhou/new-dgdrfrom
hzhou/dgdr-overrides
Feb 24, 2026
Merged

feat: add DGD overrides in profiler#6546
tedzhouhk merged 5 commits into
hzhou/new-dgdrfrom
hzhou/dgdr-overrides

Conversation

@tedzhouhk

Copy link
Copy Markdown
Contributor

Overview:

Details:

Where should the reviewer start?

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

Signed-off-by: hongkuanz <hongkuanz@nvidia.com>
@tedzhouhk
tedzhouhk requested a review from a team as a code owner February 24, 2026 21:13
@tedzhouhk
tedzhouhk requested a review from a team February 24, 2026 21:13
@github-actions github-actions Bot added the feat label Feb 24, 2026
Signed-off-by: hongkuanz <hongkuanz@nvidia.com>

@hhzhang16 hhzhang16 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.

Will approve once comments are addressed and a bit more test coverage is added!

Comment thread components/src/dynamo/profiler/profile_sla.py
Comment thread components/src/dynamo/profiler/utils/config_modifiers/protocol.py
Signed-off-by: hongkuanz <hongkuanz@nvidia.com>
Signed-off-by: hongkuanz <hongkuanz@nvidia.com>
Comment thread tests/profiler/configs/10_thorough_override_security_context.yaml Outdated

@hhzhang16 hhzhang16 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.

approved with a nit

Signed-off-by: hongkuanz <hongkuanz@nvidia.com>
@tedzhouhk
tedzhouhk merged commit 3ed7144 into hzhou/new-dgdr Feb 24, 2026
34 of 36 checks passed
@tedzhouhk
tedzhouhk deleted the hzhou/dgdr-overrides branch February 24, 2026 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants