[vs-test]: not forward routes with no-export community#1774
Merged
lguohan merged 1 commit intosonic-net:masterfrom Jun 22, 2018
Merged
[vs-test]: not forward routes with no-export community#1774lguohan merged 1 commit intosonic-net:masterfrom
lguohan merged 1 commit intosonic-net:masterfrom
Conversation
Signed-off-by: Sihui Han <[email protected]>
Collaborator
|
retest this please |
lguohan
approved these changes
Jun 22, 2018
qiluo-msft
added a commit
to qiluo-msft/sonic-buildimage
that referenced
this pull request
Jun 29, 2018
…-net#1774)" This reverts commit b37540f.
lguohan
pushed a commit
that referenced
this pull request
Jun 29, 2018
sihuihan88
pushed a commit
to sihuihan88/sonic-buildimage
that referenced
this pull request
Jul 23, 2018
sonic-net#1774)" (sonic-net#1828)" This reverts commit 6459275.
lguohan
pushed a commit
that referenced
this pull request
Jul 24, 2018
* Revert "Revert "[vs-test]: not forward routes with no-export community (#1774)" (#1828)" This reverts commit 6459275. * [no-export]: move no export test under bgp folder Signed-off-by: Sihui Han <[email protected]> * [no-export]: increase wait time to avoid random failure Signed-off-by: Sihui Han <[email protected]>
volodymyrsamotiy
added a commit
to volodymyrsamotiy/sonic-buildimage
that referenced
this pull request
Aug 26, 2021
* a72e407 - (HEAD, origin/202012) [202012][fast-reboot] Remove FLEX_COUNTER_TABLE from config_db.json before reboot (sonic-net#1774) (19 hours ago) <Vaibhav Hemant Dixit> * 205aff8 - (HEAD, origin/202012) [acl_loader]: add iptype match to the rules for dataplane acl (25 hours ago) <Guohan Lu> * 8aa2226 - Add mock support for swsscommon classes (sonic-net#1780) (26 hours ago) <Qi Luo> * d939cfb - [Show] Update the subcommands of Kdump. (sonic-net#1682) (3 days ago) <yozhao101> * 6376b5c - [route_check] Filter out VNET routes (sonic-net#1612) (3 days ago) <Volodymyr Samotiy> * ba1e457 - [show priority-group drop counters] Add user info output when user want to check PG counters and polling are disabled (sonic-net#1678) (2 weeks ago) <Andriy Yurkiv> * 4e7707c - [minigraph] Donot enable PFC watchdog for MgmtTsToR (sonic-net#1734) (2 weeks ago) <Blueve> Signed-off-by: Volodymyr Samotiy <[email protected]>
noaOrMlnx
pushed a commit
to noaOrMlnx/sonic-buildimage
that referenced
this pull request
Nov 24, 2025
…tomatically (sonic-net#1774) #### Why I did it src/sonic-gnmi ``` * e7e39d0 - (HEAD -> 202412, origin/202412) added code for 'show feature status' and 'show feature autorestart' (sonic-net#142) (21 hours ago) [Meghna Mishra] ``` #### How I did it #### How to verify it #### Description for the changelog
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.
Signed-off-by: Sihui Han [email protected]
- What I did
Add vs test to verify that the device doesn't forward routes with no-export community.
- How I did it
Establish two exabgp neighbors, and let one neighbor announce two routes, 1.1.1.1/32 (with no-export community) and 2.2.2.2/32 (without no-export community). The DUT should only announce 2.2.2.2/32 to the other neighbor
- How to verify it
Run the test.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)