From d339f86efe2d0a549d9b1130c1fc12e00b8dc9cc Mon Sep 17 00:00:00 2001 From: Mohana Date: Mon, 28 Oct 2024 02:45:28 -0700 Subject: [PATCH] Removed disabling autonegotiation and explicitly setting port speed and duplex mode as it is not part of readme (#3542) Co-authored-by: KandukuriSudheer --- feature/gribi/otg_tests/encap_frr/encap_frr_test.go | 7 ------- .../vrf_policy_driven_te/vrf_policy_driven_te_test.go | 6 ------ 2 files changed, 13 deletions(-) diff --git a/feature/gribi/otg_tests/encap_frr/encap_frr_test.go b/feature/gribi/otg_tests/encap_frr/encap_frr_test.go index 621a680fd91..80138f55dd3 100644 --- a/feature/gribi/otg_tests/encap_frr/encap_frr_test.go +++ b/feature/gribi/otg_tests/encap_frr/encap_frr_test.go @@ -258,13 +258,6 @@ func dutInterface(p *ondatra.Port, dut *ondatra.DUTDevice) *oc.Interface { i.Enabled = ygot.Bool(true) } - if p.PMD() == ondatra.PMD100GBASEFR && dut.Vendor() != ondatra.CISCO { - e := i.GetOrCreateEthernet() - e.AutoNegotiate = ygot.Bool(false) - e.DuplexMode = oc.Ethernet_DuplexMode_FULL - e.PortSpeed = oc.IfEthernet_ETHERNET_SPEED_SPEED_100GB - } - ipv4, ok := portsIPv4[id] if !ok { return nil diff --git a/feature/gribi/otg_tests/vrf_policy_driven_te/vrf_policy_driven_te_test.go b/feature/gribi/otg_tests/vrf_policy_driven_te/vrf_policy_driven_te_test.go index 136cf377d60..e8ed9a01ae0 100644 --- a/feature/gribi/otg_tests/vrf_policy_driven_te/vrf_policy_driven_te_test.go +++ b/feature/gribi/otg_tests/vrf_policy_driven_te/vrf_policy_driven_te_test.go @@ -666,12 +666,6 @@ func configureDUT(t *testing.T, dut *ondatra.DUTDevice) { for idx, a := range []attrs.Attributes{dutPort1, dutPort2, dutPort3, dutPort4, dutPort5, dutPort6, dutPort7, dutPort8} { p := portList[idx] intf := a.NewOCInterface(p.Name(), dut) - if p.PMD() == ondatra.PMD100GBASEFR && dut.Vendor() != ondatra.CISCO { - e := intf.GetOrCreateEthernet() - e.AutoNegotiate = ygot.Bool(false) - e.DuplexMode = oc.Ethernet_DuplexMode_FULL - e.PortSpeed = oc.IfEthernet_ETHERNET_SPEED_SPEED_100GB - } gnmi.Replace(t, dut, d.Interface(p.Name()).Config(), intf) }