Update py_gnmicli.py for gNMI Subscribe support
Adding support for gNMI subscribe.
Codecov Report
Merging #65 into master will not change coverage. The diff coverage is
n/a.
@@ Coverage Diff @@
## master #65 +/- ##
=======================================
Coverage 52.79% 52.79%
=======================================
Files 10 10
Lines 1197 1197
=======================================
Hits 632 632
Misses 482 482
Partials 83 83
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 64f55df...cdd72b5. Read the comment docs.
@mike-albano Can you please help to get this merge, this has fix for xpath construction with (- and /) and also subscribe request. Thanks.
Codecov Report
Merging #65 into master will increase coverage by
0.11%. The diff coverage isn/a.
@@ Coverage Diff @@
## master #65 +/- ##
==========================================
+ Coverage 52.32% 52.44% +0.11%
==========================================
Files 10 12 +2
Lines 1204 1249 +45
==========================================
+ Hits 630 655 +25
- Misses 491 510 +19
- Partials 83 84 +1
| Impacted Files | Coverage Δ | |
|---|---|---|
| gnoi/gnoi.go | 63.63% <0.00%> (-3.04%) |
:arrow_down: |
| gnoi/reset/client.go | 53.57% <0.00%> (ø) |
|
| gnoi/reset/server.go | 66.66% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 92c6d0d...965d41d. Read the comment docs.
@mike-albano Changes done. Please review. Thanks.
@mike-albano Just to make sure this is not missed.