rt-1.23 reverting skip_bgp_session_check_without_afisafi and updating…

… lookup instead of get
openconfig · Feb 28, 2025 · eb94a9c · eb94a9c
1 parent ee80a3d
commit eb94a9c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/feature/bgp/otg_tests/bgp_afi_safi_defaults/bgp_afi_safi_defaults_test.go b/feature/bgp/otg_tests/bgp_afi_safi_defaults/bgp_afi_safi_defaults_test.go
@@ -355,8 +355,14 @@ func verifyBgpCapabilities(t *testing.T, dut *ondatra.DUTDevice, afiSafiLevel st
 				if !nbr.isV4 && afiSafi == oc.BgpTypes_AFI_SAFI_TYPE_IPV4_UNICAST {
 					continue
 				}
-				t.Logf("AFI-SAFI state: %v", gnmi.Get(t, dut, statePath.Neighbor(nbr.neighborip).AfiSafi(afiSafi).State()))
-				afiSafiList = append(afiSafiList, gnmi.Get(t, dut, statePath.Neighbor(nbr.neighborip).AfiSafi(afiSafi).State()))
+				t.Logf("AFI-SAFI state: %v", gnmi.Lookup(t, dut, statePath.Neighbor(nbr.neighborip).AfiSafi(afiSafi).State()))
+				res := gnmi.Lookup(t, dut, statePath.Neighbor(nbr.neighborip).AfiSafi(afiSafi).State())
+				if res != nil {
+					val, err := res.Val()
+					if !err {
+						afiSafiList = append(afiSafiList, val)
+					}
+				}
 				t.Logf("AFI-SAFI list: %v", afiSafiList)
 			}
 			for _, cap := range afiSafiList {

diff --git a/feature/bgp/otg_tests/bgp_afi_safi_defaults/metadata.textproto b/feature/bgp/otg_tests/bgp_afi_safi_defaults/metadata.textproto
@@ -14,7 +14,7 @@ platform_exceptions: {
     bgp_extended_next_hop_encoding_leaf_unsupported: true
     bgp_global_extended_next_hop_encoding_unsupported: true
     bgp_afi_safi_wildcard_not_supported: true
-    skip_bgp_session_check_without_afisafi: true
+
   }
 }
 platform_exceptions: {