• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

openconfig / ygot / 17844674714
89%

Build:
DEFAULT BRANCH: master
Ran 19 Sep 2025 12:19AM UTC
Jobs 1
Files 84
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

03 Sep 2025 03:41PM UTC coverage: 88.728% (-0.02%) from 88.744%
17844674714

push

github

web-flow
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

New Missed Lines in Diff

Lines Coverage ∆ File
4
86.33
-0.58% ygot/diff.go
Jobs
ID Job ID Ran Files Coverage
1 17844674714.1 19 Sep 2025 12:19AM UTC 84
88.73
GitHub Action Run
Source Files on build 17844674714
  • Tree
  • List 84
  • Changed 1
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 0f8499ba on github
  • Prev Build on master (#17364242516)
  • Next Build on master (#17854680087)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc