Measure 'show ip route bgp' on VM PortChannel to test BGP graceful restart#764
Closed
qiluo-msft wants to merge 3 commits intosonic-net:masterfrom
Closed
Measure 'show ip route bgp' on VM PortChannel to test BGP graceful restart#764qiluo-msft wants to merge 3 commits intosonic-net:masterfrom
qiluo-msft wants to merge 3 commits intosonic-net:masterfrom
Conversation
Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
lguohan
reviewed
Dec 31, 2018
| po_ip_route_output = "\n".join(po_ip_route_output.split("\r\n")[1:-1]) | ||
| po_route = int(po_ip_route_output) | ||
| info["po_route"] = po_route | ||
|
|
Contributor
There was a problem hiding this comment.
I believe v5_routing_ok and v6 routing ok has the route, you do not need to do another show ip route bgp.
Contributor
Author
|
I double checked and found it is already covered by v4_routing_ok and v6_routing_ok. So no need to add this PR. |
wangxin
pushed a commit
to wangxin/sonic-mgmt
that referenced
this pull request
Oct 27, 2025
…nic-net#764) ### Description of PR The Python 3 only `docker-ptf:py3only` running ExaBGP v4 has different configuration structure and notification format. The current code does not work with ExaBGP v4. This PR contains modifications to make the `test_traffic_shift` test case work with ExaBGP v3 and v4. Announcement format for ExaBGP 3.x - ``` "message": { "update": { "attribute": { "origin": "igp", "as-path": [65200, 6666, 6667], "confederation-path": [], "local-preference": 100, "community": [[5060, 12345]] }, "announce": { "ipv4 unicast": { "10.0.0.9": { "0.0.0.0/0": {} } } } } } ``` ExaBGP v4 format - ``` "message": { "update": { "attribute": { "origin": "igp", "as-path": [65200, 6666, 6667], "confederation-path": [], "local-preference": 100, "community": [[5060, 12345]] }, "announce": { "ipv4 unicast": { "10.0.0.9": [{ "nlri": "0.0.0.0/0" } ] } } } } ``` Summary: Fixes # (issue) ### Type of change - [x] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] New Test case - [ ] Skipped for non-supported platforms - [ ] Test case improvement ### Back port request - [ ] 202012 - [ ] 202205 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [x] 202411 ### Approach #### What is the motivation for this PR? Test fails in docker-ptf Python 3 image. #### How did you do it? Modify code to setup ExaBGP v4 with new process API configuration for "dump" and handle new announcement structure. #### How did you verify/test it? Ran test in t1-lag vs test. #### Any platform specific information? NA #### Supported testbed topology if it's a new test case? NA ### Documentation NA
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
* 20b9573 2021-01-24 | [SAI]: update SAI submodule (sonic-net#775) (HEAD, origin/master, origin/HEAD) [lguohan] * 667c33d 2021-01-22 | [syncd] Comparison logic add support to LABEL attribute with higher priority (sonic-net#764) [Kamil Cudnik] * aaf5b98 2021-01-22 | [vslib]: Fix missing MACsec Create Port action (sonic-net#770) [Ze Gan] Signed-off-by: Guohan Lu <lguohan@gmail.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…onic-net#2594) * [201811 submodules] advance sairedis, swss, swss-common, utilities Submodule src/sonic-sairedis d57222a..54b02a1: > Add support for fdb_event MOVE and check fdb event oids (sonic-net#420) > [vslib] add missing port attributes for virtual switch (sonic-net#419) Submodule src/sonic-swss 584490c..62e2a20: > [fdborch] Store vlan id of FDB entry instead of port pvid into stateD… (sonic-net#759) > [portsorch] Change speed set flow (sonic-net#764) > Create egress ACL table group during the PFCWD stats list installment (sonic-net#787) Submodule src/sonic-swss-common 286ef34..2592b0c: > Add support for WarmStart::setDataCheckState() (sonic-net#242) Submodule src/sonic-utilities b44b462..d409987: > [sonic_installer] Improve error handling (sonic-net#460) > [clock] sync os time to CMOS before rebooting (sonic-net#465) > [intfstat] add intfstat script to output RIF counters (sonic-net#439) Signed-off-by: Ying Xie <ying.xie@microsoft.com> * [201811 utilties] add 201811 branch for sonic-utilities repo Submodule src/sonic-utilities d409987...ff1448c: > [sonic_installer] Improve error handling (sonic-net#460) > [clock] sync os time to CMOS before rebooting (sonic-net#465) < [sonic_installer] Improve error handling (sonic-net#460) < [clock] sync os time to CMOS before rebooting (sonic-net#465) < [intfstat] add intfstat script to output RIF counters (sonic-net#439) Signed-off-by: Ying Xie <ying.xie@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…D automatically (#25721) #### Why I did it src/sonic-platform-daemons ``` * 5d2cfce - (HEAD -> 202405, origin/202405) [ci][202405] refine pipeline to download necessary artifacts (sonic-net#764) (11 hours ago) [yijingyan2] ``` #### How I did it #### How to verify it #### Description for the changelog
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
[config] Add 'config interface mtu' command sonic-net#793 add fec configuration 'config interface fec [OPTIONS] <interface_name> <interface_fec>' sonic-net#764
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.