RT-1.23 : Added extended encoding on OTG ipv6 peers (openconfig#2938)

* Added extended encoding * added rpc yaml * added rpc yaml * added rpc yaml * fixing static error * fixing static error --------- Co-authored-by: Ram <[email protected]>
open-traffic-generator · Sep 3, 2024 · c8a951e · c8a951e
1 parent 2f7bcca
commit c8a951e
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 46 deletions.
diff --git a/feature/experimental/bgp/otg_tests/bgp_afi_safi_defaults/README.md b/feature/experimental/bgp/otg_tests/bgp_afi_safi_defaults/README.md
@@ -75,47 +75,43 @@ BGP AFI SAFI OC DEFAULTS TEST
         *   For IPv6 neighbor ensure that the IPv6 neighborship is not ESTABLISHED and
             IPv6-unicast capabilities are set to FALSE.
 
-## Config Parameter coverage
-
-*   /network-instances/network-instance/protocols/protocol/bgp/global/config/as
-*   /network-instances/network-instance/protocols/protocol/bgp/global/config/router-id
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/auth-password
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/   
-    neighbor-address
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/peer-as
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/neighbor-address
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/afi-safis/afi-safi/
-    config/enabled
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/config/
-    auth-password
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/config/
-    neighbor-address
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/config/peer-as
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/peer-group/
-    peer-group-name
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/afi-safis/
-    afi-safi/config/enabled
-*   /network-instances/network-instance/protocols/protocol/bgp/global/afi-safis/afi-safi/config/enabled
-
-
-## Telemetry Parameter coverage
-
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/session-state
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/
-    supported-capabilities
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/peer-type
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/peer-as
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/
-    supported-capabilities
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/peer-type
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/peer-as
-*   /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/local-as
-*   /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/peer-group
-
-## Protocol/RPC Parameter coverage
-
-N/A
-
-## Minimum DUT platform requirement
-
-N/A
+
+## OpenConfig Path and RPC Coverage
+
+The below yaml defines the OC paths intended to be covered by this test.  OC paths used for test setup are not listed here.
+
+```yaml
+paths:
+    ## Config Parameter coverage
+
+    /network-instances/network-instance/protocols/protocol/bgp/global/config/as:
+    /network-instances/network-instance/protocols/protocol/bgp/global/config/router-id:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/auth-password:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/neighbor-address:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/config/peer-as:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/neighbor-address:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/afi-safis/afi-safi/config/enabled:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/config/auth-password:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/config/peer-as:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/afi-safis/afi-safi/config/enabled:
+    /network-instances/network-instance/protocols/protocol/bgp/global/afi-safis/afi-safi/config/enabled:
+
+    ## Telemetry Parameter coverage
+
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/session-state:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/supported-capabilities: 
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/peer-type:
+    /network-instances/network-instance/protocols/protocol/bgp/neighbors/neighbor/state/peer-as:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/peer-type:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/peer-as:
+    /network-instances/network-instance/protocols/protocol/bgp/peer-groups/peer-group/state/local-as:
+
+rpcs:
+    gnmi:
+        gNMI.Set:
+        gNMI.Get:
+        gNMI.Subscribe:
+```
+## Minimum DUT Required
+
+vRX - Virtual Router Device
diff --git a/feature/experimental/bgp/otg_tests/bgp_afi_safi_defaults/bgp_afi_safi_defaults_test.go b/feature/experimental/bgp/otg_tests/bgp_afi_safi_defaults/bgp_afi_safi_defaults_test.go
@@ -286,7 +286,7 @@ func configureOTG(t *testing.T, otg *otg.OTG, otgPeerList []string) gosnappi.Con
 		case otgPort1V6Peer:
 			iDut1Bgp6Peer := iDut1Bgp.Ipv6Interfaces().Add().SetIpv6Name(iDut1Ipv6.Name()).Peers().Add().SetName(otgPort1V6Peer)
 			iDut1Bgp6Peer.SetPeerAddress(iDut1Ipv6.Gateway()).SetAsNumber(ateAS).SetAsType(gosnappi.BgpV6PeerAsType.EBGP)
-			iDut1Bgp6Peer.Capability().SetIpv4UnicastAddPath(true).SetIpv6UnicastAddPath(true)
+			iDut1Bgp6Peer.Capability().SetIpv4UnicastAddPath(true).SetIpv6UnicastAddPath(true).SetExtendedNextHopEncoding(true)
 			iDut1Bgp6Peer.LearnedInformationFilter().SetUnicastIpv4Prefix(true).SetUnicastIpv6Prefix(true)
 		case otgPort2V4Peer:
 			iDut2Bgp4Peer := iDut2Bgp.Ipv4Interfaces().Add().SetIpv4Name(iDut2Ipv4.Name()).Peers().Add().SetName(otgPort2V4Peer)
@@ -296,7 +296,7 @@ func configureOTG(t *testing.T, otg *otg.OTG, otgPeerList []string) gosnappi.Con
 		case otgPort2V6Peer:
 			iDut2Bgp6Peer := iDut2Bgp.Ipv6Interfaces().Add().SetIpv6Name(iDut2Ipv6.Name()).Peers().Add().SetName(otgPort2V6Peer)
 			iDut2Bgp6Peer.SetPeerAddress(iDut2Ipv6.Gateway()).SetAsNumber(dutAS).SetAsType(gosnappi.BgpV6PeerAsType.IBGP)
-			iDut2Bgp6Peer.Capability().SetIpv4UnicastAddPath(true).SetIpv6UnicastAddPath(true)
+			iDut2Bgp6Peer.Capability().SetIpv4UnicastAddPath(true).SetIpv6UnicastAddPath(true).SetExtendedNextHopEncoding(true)
 			iDut2Bgp6Peer.LearnedInformationFilter().SetUnicastIpv4Prefix(true).SetUnicastIpv6Prefix(true)
 		}
 	}