Basic PATCH action failed for CVL about arlo master branch mgmt-framework

[kun cao]

Hi, all:

I have notice that the https://github.com/project-arlo/sonic-mgmt-framework have not updated in lately 4 month, and https://github.com/project-arlo/sonic-mgmt-framework is the most active branch.

Then I have try to build the arlo  mgmt-framework master tag docker to SONiC.

The GET action is OK, but PATCH action failed. I have try two config, both failed.

1. interface's mtu

curl -X PATCH "https://10.10.38.228/restconf/data/openconfig-interfaces:interfaces/interface=Ethernet10/config/mtu" -H  "accept: application/yang-data+json" -H  "Content-Type: application/yang-data+json" -d "{  \"openconfig-interfaces:mtu\": 2000}" -k

I1222 22:38:59.473102    2821 common_app.go:471] Processing Table key Ethernet10
I1222 22:38:59.473121    2821 common_app.go:484] Processing Table row "mtu": "2000"
I1222 22:38:59.473353    2821 common_app.go:500] Entry already exists hence modifying it.
I1222 22:38:59.473384    2821 common_app.go:726] Returning Table Row "mtu": "2000"
E1222 22:38:59.474153    2821 util.go:298] [CVL] : Syntax validation failed. Error - { 1020  []     }
E1222 22:38:59.474213    2821 util.go:298] [CVL] : ValidateEditConfig() failed , {TableName: ErrCode:1020 CVLErrDetails: Keys:[] Value: Field: Msg: ConstraintErrMsg: ErrAppTag:}
E1222 22:38:59.474254    2821 db.go:596] doCVL: CVL Failure: 1020
E1222 22:38:59.474287    2821 db.go:600] doCVL: 1 1
E1222 22:38:59.474315    2821 common_app.go:508] UPDATE case - d.ModEntry() failure
I1222 22:38:59.474334    2821 common_app.go:402] Process update fail. cmnAppCRUCommonDbOpn error:Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
E1222 22:38:59.474426    2821 common_app.go:160] Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
E1222 22:38:59.474716    2821 handler.go:71] [REST-2] Translib error tlerr.TranslibCVLFailure - Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
I1222 22:38:59.475047    2821 handler.go:91] [REST-2] Sending response 500, type=application/yang-data+json, size=93
I1222 22:38:59.475079    2821 handler.go:100] [REST-2] data={"ietf-restconf:errors":{"error":[{"error-type":"application","error-tag":"invalid-value"}]}}
I1222 22:38:59.475114    2821 router.go:441] [REST-2] PatchOpenconfigInterfacesInterfacesInterfaceConfigMtu took 7.959886ms

2. interface's description
curl -X PATCH "https://10.10.38.228/restconf/data/openconfig-interfaces:interfaces/interface=Ethernet10/config/description" -H  "accept: application/yang-data+json" -H  "Content-Type: application/yang-data+json" -d "{  \"openconfig-interfaces:description\": \"test123\"}"  -k

I1222 22:38:59.473083    2821 common_app.go:465] Found table entry in yang to DB map
I1222 22:38:59.473102    2821 common_app.go:471] Processing Table key Ethernet10
I1222 22:38:59.473121    2821 common_app.go:484] Processing Table row "mtu": "2000"
I1222 22:38:59.473353    2821 common_app.go:500] Entry already exists hence modifying it.
I1222 22:38:59.473384    2821 common_app.go:726] Returning Table Row "mtu": "2000"
E1222 22:38:59.474153    2821 util.go:298] [CVL] : Syntax validation failed. Error - { 1020  []     }
E1222 22:38:59.474213    2821 util.go:298] [CVL] : ValidateEditConfig() failed , {TableName: ErrCode:1020 CVLErrDetails: Keys:[] Value: Field: Msg: ConstraintErrMsg: ErrAppTag:}
E1222 22:38:59.474254    2821 db.go:596] doCVL: CVL Failure: 1020
E1222 22:38:59.474287    2821 db.go:600] doCVL: 1 1
E1222 22:38:59.474315    2821 common_app.go:508] UPDATE case - d.ModEntry() failure
I1222 22:38:59.474334    2821 common_app.go:402] Process update fail. cmnAppCRUCommonDbOpn error:Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
E1222 22:38:59.474426    2821 common_app.go:160] Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
E1222 22:38:59.474716    2821 handler.go:71] [REST-2] Translib error tlerr.TranslibCVLFailure - Translib Redis Error: CVL Failure: 1,020: { 1,020  []     }
I1222 22:38:59.475047    2821 handler.go:91] [REST-2] Sending response 500, type=application/yang-data+json, size=93
I1222 22:38:59.475079    2821 handler.go:100] [REST-2] data={"ietf-restconf:errors":{"error":[{"error-type":"application","error-tag":"invalid-value"}]}}
I1222 22:38:59.475114    2821 router.go:441] [REST-2] PatchOpenconfigInterfacesInterfacesInterfaceConfigMtu took 7.959886ms

 I try to gdb the rest_server, and found the error is from the 3th party library yparser.

if errObj := c.py.ValidateSyntax(data, depData); errObj.ErrCode != yparser.YP_SUCCESS.

Could any one give me some advice to pull through the problem? 

Thanks a lot!

kun

[Ben Gale]

Kun,

Please don't use project-arlo - this is a "semi-private fork", and not really for Community use.

Please use Community Master (https://github.com/Azure/sonic-mgmt-framework)

Ben

[kun cao]

hi, Ben:

OK, thanks! 

But I have a question, the Community Master is imcompleted, does the Community Master still being evoluated?

If will be evoluated, it will evolute from project-arlo's mgmt-framework or other branches?

 

Kun

在 2020年4月29日星期三 UTC+8下午7:18:44，Ben Gale写道：

[Ben Gale]

Yes - the Management Framework will continue to evolve over the next few SONiC releases - in fact the latest set of design updates can be found here - https://github.com/Azure/SONiC/pull/550 (reviewed at the April 14 SONiC Community call) - these will appear as code PRs over the next few weeks.

Thanks.