zebra: Add support for json output in srv6 locator detail command#9899
Conversation
Drumato
force-pushed
the
zebra-srv6-locator-detail-json-support
branch
from
a2c5e75 to
ed75a03
Compare
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests debian 10 amd64 part 9: Failed (click for details)Topotests debian 10 amd64 part 9: No useful log foundSuccessful on other platforms/tests
|
Outdated results 💚Basic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 i386 part 9: Failed (click for details)Topotests Ubuntu 18.04 i386 part 9: No useful log foundTopotests Ubuntu 18.04 amd64 part 9: Failed (click for details)Topotests Ubuntu 18.04 amd64 part 9: No useful log foundSuccessful on other platforms/tests
|
|
ci:rerun |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests debian 10 amd64 part 9: Failed (click for details)Topotests debian 10 amd64 part 9: No useful log foundSuccessful on other platforms/tests
|
|
ci:rerun |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 i386 part 9: Failed (click for details)Topotests Ubuntu 18.04 i386 part 9: No useful log foundSuccessful on other platforms/tests
|
|
ci:rerun |
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1236/ This is a comment from an automated CI system. |
|
@donaldsharp Could you take a look at this PR? This PR is my first-time contribution to FRR, |
Drumato
force-pushed
the
zebra-srv6-locator-detail-json-support
branch
from
ed75a03 to
a621bed
Compare
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 amd64 part 4: Failed (click for details)Topotests Ubuntu 18.04 amd64 part 4: No useful log foundSuccessful on other platforms/tests
|
|
ci:rerun |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDTest incomplete. See below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: IncompleteUbuntu 18.04 arm8 build: Incomplete(check logs for details)Ubuntu 16.04 arm8 build: Incomplete(check logs for details)Successful on other platforms/tests
|
|
ci:rerun |
1 similar comment
|
ci:rerun |
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1822/ This is a comment from an automated CI system. |
|
@ton31337 Thanks for reviewing my PR! |
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1823/ This is a comment from an automated CI system. |
Drumato
force-pushed
the
zebra-srv6-locator-detail-json-support
branch
3 times, most recently
from
be2fd5c to
7c6c2fc
Compare
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1826/ This is a comment from an automated CI system. Warnings Generated during build:Checkout code: Successful with additional warnings |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 i386 part 9: Failed (click for details)Topotests Ubuntu 18.04 i386 part 9: No useful log foundSuccessful on other platforms/tests
Warnings Generated during build:Checkout code: Successful with additional warningsTopotests Ubuntu 18.04 i386 part 9: Failed (click for details)Topotests Ubuntu 18.04 i386 part 9: No useful log found |
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1827/ This is a comment from an automated CI system. |
|
@ton31337 thanks for re-reviewing🙇♀️ |
|
Please squash into a single commit here and seems good... |
Drumato
force-pushed
the
zebra-srv6-locator-detail-json-support
branch
from
7c6c2fc to
ac3bb00
Compare
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1835/ This is a comment from an automated CI system. |
Signed-off-by: Yamato Sugawara <yamato.sugawara@linecorp.com>
Drumato
force-pushed
the
zebra-srv6-locator-detail-json-support
branch
from
ac3bb00 to
559f4b2
Compare
Continuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-1842/ This is a comment from an automated CI system. |
4 participants
Add the json output to
show-srv6-locator-detailcommand.At almost cases, these fields may be filled by zero but its not bad for future extensibility.
here's a sample output that's generated from
vtysh -c 'sh segment-routing srv6 locator loc1 detail json' | jqin srv6_locator topotest.{ "name": "loc1", "prefix": "2001:db8:1:1::/64", "blockBitsLength": 40, "nodeBitsLength": 24, "functionBitsLength": 0, "argumentBitsLength": 0, "algoNum": 0, "statusUp": true, "chunks": [ { "prefix": "2001:db8:1:1::/64", "blockBitsLength": 0, "nodeBitsLength": 0, "functionBitsLength": 0, "argumentBitsLength": 0, "keep": 0, "proto": "system", "instance": 0, "sessionId": 0 } ] }Signed-off-by: Drumato drumato43@gmail.com