Skip to content

Fix: 5968806: Use Option<T> for the dpf_enabled flag to avoid sending default value to server.#537

Open
abvarshney-nv wants to merge 1 commit intoNVIDIA:mainfrom
abvarshney-nv:dpf-status-admin-cli
Open

Fix: 5968806: Use Option<T> for the dpf_enabled flag to avoid sending default value to server.#537
abvarshney-nv wants to merge 1 commit intoNVIDIA:mainfrom
abvarshney-nv:dpf-status-admin-cli

Conversation

@abvarshney-nv
Copy link
Contributor

Description

Use Option for the dpf_enabled flag to avoid sending default value to server.

  1. Any EM which is already created and dpf_enabled is not updated will be updated as dpf_enabled false.
  2. If during add, dpf_enabled is not mentioned, will be updated as false.
  3. if during patching, dpf_enabled is not provided, the value will remain unchanged.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

NVBUG-5968806

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

… default value to server.

1. Any EM which is already created and dpf_enabled is not updated will
be updated as dpf_enabled false.
2. If during add, dpf_enabled is not mentioned, will be updated as
false.
3. if during patching, dpf_enabled is not provided, the value will
remain unchanged.
@abvarshney-nv abvarshney-nv requested a review from a team as a code owner March 12, 2026 09:43
@github-actions
Copy link

🔐 TruffleHog Secret Scan

No secrets or credentials found!

Your code has been scanned for 700+ types of secrets and credentials. All clear! 🎉

🔗 View scan details

🕐 Last updated: 2026-03-12 09:45:18 UTC | Commit: 858c200

@github-actions
Copy link

🛡️ Vulnerability Scan

🚨 Found 74 vulnerability(ies)
📊 vs main: 74 (no change)

Severity Breakdown:

  • 🔴 Critical/High: 74
  • 🟡 Medium: 0
  • 🔵 Low/Info: 0

🔗 View full details in Security tab

🕐 Last updated: 2026-03-12 09:45:21 UTC | Commit: 858c200

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