openconfig / ygot / 17844674714

89%

push

github

Add OverrideLeafList support in ygot diff. (#1046)

* Add DeleteWithUpdateLeafList support in ygot diff.

Implemented handling for DeleteWithUpdateLeafList operation in diff.go
Added corresponding test cases in diff_test.go
Tested locally to ensure correct functionality and pushed the generated Diff config in Arista FakeFabric (Google Internal Tool).

Details:
Explicitly delete and then update leaf-list, which is different and exists in both orig and modified.
For example, if the original path is x/y/[a, b] and the modified leaf-list is x/y/[b, c],
Then resulting notification will contain:

Delete x/y/[a, b]
Update x/y/[b, c]

If this is false, then the leaf-list will be updated directly to the new value
without the delete step.
This is required because if the leaf-list is updated directly,
the resulting notification will be:
Update x/y/[b, c]

This will cause the new value to be appended in the OC, but the original value
will still be present in the OC. Resultant OC in the device will be x/y/[a, b, c].

* add newline

* rename to OverrideLeafList

* correct the delete op comment

* correct the comment

* ignore binary type

* fix tests

* ignore binary not uint8

* improve code

13 of 17 new or added lines in 1 file covered. (76.47%)

13995 of 15773 relevant lines covered (88.73%)

485.31 hits per line