OTG Test - RT-1.4: BGP Graceful Restart #1509
Conversation
…profiles into otg-rt1.4
-updated README -updated the OTG configuration -added gnoi commands
-corrected the direction of traffic for 1.4.7 -- 1.4.10
|
@sachendras could you please review the corrections I've made to the README file. I'm assuming we would want to run 1.4.5 for both IBGP and EBGP. Also for tests 1.4.7 through 1.4.10 when testing EBGP I'm assuming the direction of traffic is ATE port2 - ATE Port1 (EBGP routes). In this case the notification will be sent from DUT port1 |
| } | ||
|
|
||
| func TestBGPPGracefulRestart(t *testing.T) { | ||
| cases := []struct { |
There was a problem hiding this comment.
Move these cases before the RT-1.4.1 test so that they are defined closer to where they are used.
feature/bgp/gracefulrestart/otg_tests/bgp_graceful_restart_test/bgp_graceful_restart_test.go
Show resolved
Hide resolved
feature/bgp/gracefulrestart/otg_tests/bgp_graceful_restart_test/bgp_graceful_restart_test.go
Outdated
Show resolved
Hide resolved
feature/bgp/gracefulrestart/otg_tests/bgp_graceful_restart_test/bgp_graceful_restart_test.go
Outdated
Show resolved
Hide resolved
feature/bgp/gracefulrestart/otg_tests/bgp_graceful_restart_test/bgp_graceful_restart_test.go
Outdated
Show resolved
Hide resolved
| pgGrV4.RestartTime = ygot.Uint16(grRestartTime) | ||
| pgGrV4.StaleRoutesTime = ygot.Uint16(grStaleRouteTime) |
There was a problem hiding this comment.
These (RestartTime and StaleRoutesTime) are indicated in the README as set at the 'global' level. (I believe the existing ATE test does do that.) Might also imply changes in checkBgpGRConfig?
There was a problem hiding this comment.
If we do wind up sending traffic from both ATE interfaces, do we want to rename config in general to reflect IBGP vs. EBGP instead? (Eg. srcEth -> iBGPEth maybe).
OTG Test conversion of RT-1.4: BGP Graceful Restart
Resolves #580
The ACL sections were replaced by the "Initiate graceful restart" in the case of DUT helper and "gnoi restart pid" in case of DUT restart initiator