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

[YANG] leaf type missing SonicDPU in sonic-device_metadata.yang #21111

Closed
wen587 opened this issue Dec 10, 2024 · 1 comment · Fixed by #21520
Closed

[YANG] leaf type missing SonicDPU in sonic-device_metadata.yang #21111

wen587 opened this issue Dec 10, 2024 · 1 comment · Fixed by #21520
Assignees
Labels
NVIDIA Request for 202411 Branch Triaged this issue has been triaged YANG YANG model related changes

Comments

@wen587
Copy link
Contributor

wen587 commented Dec 10, 2024

Description

Steps to reproduce the issue:

  1. Checkout json file for dpu and yang validation failed https://github.com/sonic-net/sonic-buildimage/blob/master/src/sonic-config-engine/tests/sample_output/t1-smartswitch-dpu.json
    Introduced PR: [smart-switch] Extend config generator for t1-smartswitch topology. #19056
libyang[0]: Value "SonicDpu" does not satisfy the constraint "ToRRouter|LeafRouter|SpineChassisFrontendRouter|ChassisBackendRouter|ASIC|MgmtToRRouter|SpineRouter|BackEndToRRouter|BackEndLeafRouter|EPMS|MgmtTsToR|BmcMgmtToRRouter|not-provisioned" (range, length, or pattern). (path: /sonic-device_metadata:sonic-device_metadata/DEVICE_METADATA/localhost/type)
sonic_yang(3):Data Loading Failed:Value "SonicDpu" does not satisfy the constraint "ToRRouter|LeafRouter|SpineChassisFrontendRouter|ChassisBackendRouter|ASIC|MgmtToRRouter|SpineRouter|BackEndToRRouter|BackEndLeafRouter|EPMS|MgmtTsToR|BmcMgmtToRRouter|not-provisioned" (range, length, or pattern).

Describe the results you received:

Fail YANG

Describe the results you expected:

Pass YANG

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@gpunathilell
Copy link
Contributor

@wen587 The Request for 202411 branch was removed from the fix PR, as DPUs will still be in dark mode in 202411, please re-add it if required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NVIDIA Request for 202411 Branch Triaged this issue has been triaged YANG YANG model related changes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants