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

[DPB] Port Breakout Fails with "sonic_yang(3):All Keys are not parsed in CRM" #7956

Closed
alexrallen opened this issue Jun 23, 2021 · 3 comments
Labels
Port Breakout 🔱 YANG YANG model related changes

Comments

@alexrallen
Copy link
Contributor

#7195 introduced more potential leaves in the CRM Yang model, particularly...

mpls_inseg_high_threshold
mpls_inseg_low_threshold
mpls_inseg_threshold_type

and

mpls_nexthop_high_threshold
mpls_nexthop_low_threshold
mpls_nexthop_threshold_type

These needed to be added to the Yang model as it is currently causing Dynamic Port Breakout to fail due to not matching on the new fields.

Error:

root@r-bulldog-03:/home/admin# config int break Ethernet0 4x25G[10G]
Do you want to Breakout the port, continue? [y/N]: y

Running Breakout Mode : 1x100G[50G,40G,25G,10G]
Target Breakout Mode : 4x25G[10G]

Ports to be deleted :
 {
    "Ethernet0": "100000"
}
Ports to be added :
 {
    "Ethernet0": "25000",
    "Ethernet1": "25000",
    "Ethernet2": "25000",
    "Ethernet3": "25000"
}

After running Logic to limit the impact

Final list of ports to be deleted :
 {
    "Ethernet0": "100000"
}
Final list of ports to be added :
 {
    "Ethernet0": "25000",
    "Ethernet1": "25000",
    "Ethernet2": "25000",
    "Ethernet3": "25000"
}
Note: Below table(s) have no YANG models:
FEATURE, KDUMP, SNMP, SNMP_COMMUNITY, ZTP,
sonic_yang(3):All Keys are not parsed in CRM
dict_keys(['mpls_inseg_high_threshold', 'mpls_inseg_low_threshold', 'mpls_inseg_threshold_type', 'mpls_nexthop_high_threshold', 'mpls_nexthop_low_threshold', 'mpls_nexthop_threshold_type'])
sonic_yang(3):exceptionList:[]
sonic_yang(3):Data Loading Failed:All Keys are not parsed in CRM
dict_keys(['mpls_inseg_high_threshold', 'mpls_inseg_low_threshold', 'mpls_inseg_threshold_type', 'mpls_nexthop_high_threshold', 'mpls_nexthop_low_threshold', 'mpls_nexthop_threshold_type'])
Data Loading Failed
All Keys are not parsed in CRM
dict_keys(['mpls_inseg_high_threshold', 'mpls_inseg_low_threshold', 'mpls_inseg_threshold_type', 'mpls_nexthop_high_threshold', 'mpls_nexthop_low_threshold', 'mpls_nexthop_threshold_type'])
ConfigMgmt Class creation failed
Failed to break out Port. Error: Failed to load the config. Error: ConfigMgmtDPB Class creation failed
@praveen-li
Copy link
Collaborator

praveen-li commented Jun 24, 2021

The fix is to add new fields in CRM yang model...
as per #7195

Should be assigned to @qbdwlr

@qbdwlr
Copy link
Contributor

qbdwlr commented Jun 24, 2021

@praveen-li CRM yang models were originally part of #7195, but were removed due to yang sub-group review scheduling. init_cfg.json.j2 changes here should have been moved to the new yang model PR #7881.

@anshuv-mfst anshuv-mfst added the YANG YANG model related changes label Jun 24, 2021
@liat-grozovik
Copy link
Collaborator

@smaheshm can you please help to take it with the owner of #7195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Port Breakout 🔱 YANG YANG model related changes
Projects
None yet
Development

No branches or pull requests

5 participants