-
Notifications
You must be signed in to change notification settings - Fork 727
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TestGap] New tests to cover VNET route advertisement. #14666
[TestGap] New tests to cover VNET route advertisement. #14666
Conversation
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
for vnet in routes: | ||
for prefix in routes[vnet]: | ||
route = f'{prefix}/{prefix_mask}' | ||
result = self.vxlan_test_setup['t2']['host'].get_route(route) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems only verified on one T2? Should we check all uplink neighbors? @siqbal1986
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the advertisement is happening, then it either reach all T2s or none of them. The get_route operation is slow and time consuming. In order to speed up the test, i checked only one t2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@StormLiangMS I have made the changes, now the test is looking at the T2s.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@StormLiangMS please review and approve so that this can be merged
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
fixed indentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What is the motivation for this PR? How did you do it? Create the BGP profile Create a VNET routes. check neighbor bgp routes to verify the advertisements. The following tests are performed for Both V4 and V6 routes. Step Goal Expected results Create a tunnel route and advertise the tunnel route to all neighbor without community id BGP ALL BGP neighbors can recieve the advertised BGP routes Create a tunnel route and advertise the tunnel route to all neighbor with community id BGP ALL BGP neighbors can recieve the advertised BGP routes with community id Update a tunnel route and advertise the tunnel route to all neighbor with new community id BGP ALL BGP neighbors can recieve the advertised BGP routes with new community id Create a tunnel route and advertise the tunnel route to all neighbor with BGP profile, but create the profile later BGP ALL BGP neighbors can recieve the advertised BGP routes without community id first, after the profile table created, the community id would be added and all BGP neighbors can recieve this update and associate the community id with the route Delete a tunnel route BGP ALL BGP neighbors can remove the previously advertised BGP routes Create 400 tunnel routes and advertise all tunnel routes to all neighbor with community id BGP scale ALL BGP neighbors can recieve 400 advertised BGP routes with community id and record the time Updat BGP_PROFILE_TABLE with new community id for 400 tunnel routes and advertise all tunnel routes to all neighbor with new community id BGP scale ALL BGP neighbors can recieve 400 advertised BGP routes with new community id and record the time How did you verify/test it? image Any platform specific information? These scale tests are set to un with 400 routes. Altough I have ran these tests with 4k routes without any problem, but that takes the test run time to around 40 minutes. Supported testbed topology if it's a new test case? T1 Cisco, T1 Mlnx, VS
What is the motivation for this PR? How did you do it? Create the BGP profile Create a VNET routes. check neighbor bgp routes to verify the advertisements. The following tests are performed for Both V4 and V6 routes. Step Goal Expected results Create a tunnel route and advertise the tunnel route to all neighbor without community id BGP ALL BGP neighbors can recieve the advertised BGP routes Create a tunnel route and advertise the tunnel route to all neighbor with community id BGP ALL BGP neighbors can recieve the advertised BGP routes with community id Update a tunnel route and advertise the tunnel route to all neighbor with new community id BGP ALL BGP neighbors can recieve the advertised BGP routes with new community id Create a tunnel route and advertise the tunnel route to all neighbor with BGP profile, but create the profile later BGP ALL BGP neighbors can recieve the advertised BGP routes without community id first, after the profile table created, the community id would be added and all BGP neighbors can recieve this update and associate the community id with the route Delete a tunnel route BGP ALL BGP neighbors can remove the previously advertised BGP routes Create 400 tunnel routes and advertise all tunnel routes to all neighbor with community id BGP scale ALL BGP neighbors can recieve 400 advertised BGP routes with community id and record the time Updat BGP_PROFILE_TABLE with new community id for 400 tunnel routes and advertise all tunnel routes to all neighbor with new community id BGP scale ALL BGP neighbors can recieve 400 advertised BGP routes with new community id and record the time How did you verify/test it? image Any platform specific information? These scale tests are set to un with 400 routes. Altough I have ran these tests with 4k routes without any problem, but that takes the test run time to around 40 minutes. Supported testbed topology if it's a new test case? T1 Cisco, T1 Mlnx, VS
Description of PR
Summary:
Fixes # (issue)
Added new tests to cover the previously untested scenario. These tests create VNET Vxlan ECMP routes and then chekc if they are advertised to the upstream neighbors.
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
Create the BGP profile
Create a VNET routes.
check neighbor bgp routes to verify the advertisements.
The following tests are performed for Both V4 and V6 routes.
How did you verify/test it?
Any platform specific information?
These scale tests are set to un with 400 routes. Altough I have ran these tests with 4k routes without any problem, but that takes the test run time to around 40 minutes.
Supported testbed topology if it's a new test case?
T1 Cisco, T1 Mlnx, VS
Documentation