Closed renukamanavalan closed 2 years ago
I have to do some updates to YANG models to get over.
Does it mean that you don't have the problem anymore? I just want to understand, if it has been already fixed or we should fix it.
notify: @pshulik
I took the version of 11/25 applied your changes and got failure.
admin@sonic:~$ sudo config apply-patch ./dummy.json-patch
Patch Applier: Patch application starting.
Patch Applier: Patch: [{"op": "replace", "path": "/PORT/Ethernet124/admin_status", "value": "down"}]
Patch Applier: Validating patch is not making changes to tables without YANG models.
Patch Applier: Getting current config db.
Patch Applier: Simulating the target full config after applying the patch.
Patch Applier: Validating target config according to YANG models.
Note: Below table(s) have no YANG models:
DEBUG_COUNTER, DEBUG_COUNTER_DROP_REASON, KDUMP, SNMP, SNMP_COMMUNITY, TELEMETRY, XCVRD_LOG,
sonic_yang(3):All Keys are not parsed in SFLOW_COLLECTOR
dict_keys(['Collector1'])
sonic_yang(3):exceptionList:["'collector_vrf'"]
sonic_yang(3):Data Loading Failed:All Keys are not parsed in SFLOW_COLLECTOR
dict_keys(['Collector1'])
Failed to apply patch
Usage: config apply-patch [OPTIONS] PATCH_FILE_PATH
Try "config apply-patch -h" for help.
Error: Given patch is not valid because it will result in an invalid config
admin@sonic:~$ cat ./dummy.json-patch
[
{ "op": "replace", "path": "/PORT/Ethernet124/admin_status", "value": "down" }
]
@renukamanavalan Some problems you mentioned are already fixed, some I have fixed here: https://github.com/Azure/sonic-buildimage/pull/9517
@renukamanavalan Could you check if we can close this issue?
AddRack works w/o hack. Hence close.
Description
Running "sudo config apply-patch ..." in latest (as of 11/15) master image applied in SONiC switches fails with inability to parse config for given yang models. I have to do some updates to YANG models to get over.
Steps to reproduce the issue
Describe the results you received
Describe the results you expected
Additional information you deem important (e.g. issue happens only occasionally)
The list of files updated and the diff are as below.
Output of
show version
yang_updated.tar.gz