[NETCONF-197] PATCH merge operation does nothing Created: 17/May/16  Updated: 15/Mar/19  Resolved: 05/Aug/16

Status: Resolved
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: None

Type: Bug
Reporter: Vratko Polak Assignee: Ivan Hrasko
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 5897

 Description   

This is a Beryllium bug, similar to NETCONF-185. Here is what happens when launching curl commands from Bash (ODL has odl-restconf and odl-clustering-test-app features installed).

$ curl -u'admin:admin' -X PATCH -H "Content-Type:application/yang.patch+json" -d '{"ietf-restconf:yang-patch":{"patch-id":"0","edit":[{"edit-id":"0","operation":"replace","target":"/car:car-entry[car:id='"'"'0'"'"']","value":{"car:car-entry":[

{"id":"0"}

]}}]}' 127.0.0.1:8181/restconf/config/car:cars ;echo

{"ietf-yang-patch:yang-patch-status":{"patch-id":"0","ok":null}}

$ curl -u'admin:admin' 127.0.0.1:8181/restconf/config/car:cars ;echo

{"cars":{"car-entry":[

{"id":"0"}

]}}

$ curl -u'admin:admin' -X PATCH -H "Content-Type:application/yang.patch+json" -d '{"ietf-restconf:yang-patch":{"patch-id":"0","edit":[{"edit-id":"0","operation":"merge","target":"/car:car-entry[car:id='"'"'1'"'"']","value":{"car:car-entry":[

{"id":"1"}

]}}]}' 127.0.0.1:8181/restconf/config/car:cars ;echo

{"ietf-yang-patch:yang-patch-status":{"patch-id":"0","ok":null}}

$ curl -u'admin:admin' 127.0.0.1:8181/restconf/config/car:cars ;echo

{"cars":{"car-entry":[

{"id":"0"}

]}}

I was expecting both

{"id":"1"}

and

{"id":"0"}

entries to be present in the final GET response.



 Comments   
Comment by Ivan Hrasko [ 11/Jul/16 ]

Patch: https://git.opendaylight.org/gerrit/#/c/41636/

Comment by Ivan Hrasko [ 19/Jul/16 ]

Merged to Be, needs port to Boron.

Comment by Ivan Hrasko [ 27/Jul/16 ]

Cherry pick to boron old Restconf:
https://git.opendaylight.org/gerrit/#/c/42461

Comment by Ivan Hrasko [ 29/Jul/16 ]

Merged to Boron

Comment by Ivan Hrasko [ 05/Aug/16 ]

patch for new restconf has been merged to master:
https://git.opendaylight.org/gerrit/#/c/41833/

Generated at Wed Feb 07 20:14:24 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.