Skip to content

Commit

Permalink
Update Readme per PR#1538 changes
Browse files Browse the repository at this point in the history
  • Loading branch information
arvbaska1 authored and self-maurya committed Dec 27, 2024
1 parent 3a3d098 commit 38377ee
Show file tree
Hide file tree
Showing 4 changed files with 54 additions and 3 deletions.
51 changes: 51 additions & 0 deletions feature/bgp/prefixlimit/ate_tests/bgp_prefix_limit_test/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
# RT-1.5: BGP Prefix Limit

## Summary

BGP Prefix Limit

## Procedure

* Configure eBGP session between ATE port-1 and DUT port-1,with an Accept-route all import-policy/export-policy under the BGP peer-group AFI/SAFI.
* With maximum prefix limits of unlimited, and N.
* Advertise prefixes of `limit - 1`, `limit`, `limit + 1`. Validate
session state meets expected value at ATE.
* Ensure that DUT marks session as prefix-limit exceeded for limit+1
prefixes.
* Advertise prefixes to exceed configured limit, and to `limit - 1` following
session teardown, ensure session is re-established per the restart timer
(with ATE session marked as passive).
* With maximum-prefix warning-only configured, ensure that the routes that
were sent prior to the max-prefix being exceeded are retained in the routing
table by forwarding traffic to `prefix{0..n-1}` and `prefix{n}` where n is
the maximum prefix limit configured.

## Config Parameter coverage

For prefixes:

* /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/
* /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor

Parameters:

* afi-safis/afi-safi/ipv4-unicast/prefix-limit/config/max-prefixes
* afi-safis/afi-safi/ipv4-unicast/prefix-limit/config/restart-timer

## Telemetry Parameter coverage

* TODO: afi-safis/afi-safi/ipv\[46\]-unicast/prefix-limit/state/restart-timer
* TODO:
afi-safis/afi-safi/ipv\[46\]-unicast/prefix-limit/state/warning-threshold-pct
* TODO:
afi-safis/afi-safi/ipv\[46\]-unicast/prefix-limit/state/max-prefix-limit
* TODO:
afi-safis/afi-safi/ipv\[46\]-unicast/prefix-limit/state/prefix-limit-exceeded

## Protocol/RPC Parameter coverage

N/A

## Minimum DUT platform requirement

vRX
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ BGP Prefix Limit

## Procedure

* Configure eBGP session between ATE port-1 and DUT port-1,with an Accept-route all import-policy/export-policy under the neighbor AFI/SAFI.
* Configure eBGP session between ATE port-1 and DUT port-1,with an Accept-route all import-policy/export-policy under the BGP peer-group AFI/SAFI.
* With maximum prefix limits of unlimited, and N.
* Advertise prefixes of `limit - 1`, `limit`, `limit + 1`. Validate
session state meets expected value at ATE.
Expand Down
2 changes: 1 addition & 1 deletion feature/bgp/tests/local_bgp_test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The local\_bgp\_test brings up two OpenConfig controlled devices and tests that
* Disconnected between them.
* Verify BGP neighbor parameters

Enable an Accept-route all import-policy/export-policy for eBGP session under the neighbor AFI/SAFI.
Enable an Accept-route all import-policy/export-policy for eBGP session under the BGP peer-group AFI/SAFI.


This test is suitable for running in a KNE environment.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ BGP Keepalive and HoldTimer Configuration Test

* Establish eBGP sessions as follows between ATE and DUT
* The DUT has eBGP peering with ATE port 1 and ATE port 2.
* Enable an Accept-route all import-policy/export-policy under the neighbor AFI/SAFI.
* Enable an Accept-route all import-policy/export-policy under the BGP peer-group AFI/SAFI.
* The first pair is called the "source" pair, and the second the "destination" pair

* Validate BGP session state on DUT using telemetry.
Expand Down

0 comments on commit 38377ee

Please sign in to comment.