TE-1.1 static_arp_test.go (openconfig#2524)

* added flow control * Added deviation --------- Co-authored-by: Arul Kumar Sekar <128646204+arulkumarsekar@users.noreply.github.com>
aristanetworks · Jan 11, 2024 · 552d345 · 552d345
1 parent 722c554
commit 552d345
Show file tree

Hide file tree

Showing 7 changed files with 80 additions and 37 deletions.
diff --git a/feature/interface/staticarp/ate_tests/static_arp_test/metadata.textproto b/feature/interface/staticarp/ate_tests/static_arp_test/metadata.textproto
@@ -13,6 +13,14 @@ platform_exceptions: {
     ipv4_missing_enabled: true
   }
 }
+platform_exceptions: {
+  platform: {
+    vendor: JUNIPER
+  }
+  deviations: {
+    enable_flowctrl_flag: true
+  }
+}
 platform_exceptions: {
   platform: {
     vendor: NOKIA

diff --git a/feature/interface/staticarp/ate_tests/static_arp_test/static_arp_test.go b/feature/interface/staticarp/ate_tests/static_arp_test/static_arp_test.go
@@ -123,6 +123,9 @@ func configInterfaceDUT(t *testing.T, p *ondatra.Port, me, peer *attrs.Attribute
 	if me.MAC != "" {
 		e := i.GetOrCreateEthernet()
 		e.MacAddress = ygot.String(me.MAC)
+		if deviations.EnableFlowctrlFlag(dut) {
+			e.EnableFlowControl = ygot.Bool(true)
+		}
 	}
 
 	s := i.GetOrCreateSubinterface(0)

diff --git a/feature/interface/staticarp/otg_tests/static_arp_test/metadata.textproto b/feature/interface/staticarp/otg_tests/static_arp_test/metadata.textproto
@@ -13,6 +13,14 @@ platform_exceptions: {
     ipv4_missing_enabled: true
   }
 }
+platform_exceptions: {
+  platform: {
+    vendor: JUNIPER
+  }
+  deviations: {
+    enable_flowctrl_flag: true
+  }
+}
 platform_exceptions: {
   platform: {
     vendor: NOKIA

diff --git a/feature/interface/staticarp/otg_tests/static_arp_test/static_arp_test.go b/feature/interface/staticarp/otg_tests/static_arp_test/static_arp_test.go
@@ -127,6 +127,9 @@ func configInterfaceDUT(t *testing.T, p *ondatra.Port, me, peer *attrs.Attribute
 	if me.MAC != "" {
 		e := i.GetOrCreateEthernet()
 		e.MacAddress = ygot.String(me.MAC)
+		if deviations.EnableFlowctrlFlag(dut) {
+			e.EnableFlowControl = ygot.Bool(true)
+		}
 	}
 
 	s := i.GetOrCreateSubinterface(0)

diff --git a/internal/deviations/deviations.go b/internal/deviations/deviations.go
@@ -730,3 +730,8 @@ func StaticRouteNextHopInterfaceRefUnsupported(dut *ondatra.DUTDevice) bool {
 func SkipStaticNexthopCheck(dut *ondatra.DUTDevice) bool {
 	return lookupDUTDeviations(dut).GetSkipStaticNexthopCheck()
 }
+
+// EnableFlowctrlFlag returns if device needs set leaf specific enable flag.
+func EnableFlowctrlFlag(dut *ondatra.DUTDevice) bool {
+	return lookupDUTDeviations(dut).GetEnableFlowctrlFlag()
+}
diff --git a/proto/metadata.proto b/proto/metadata.proto
@@ -389,6 +389,9 @@ message Metadata {
     // Devices which does not support nexthop index state
     // Juniper: b/304729237
     bool skip_static_nexthop_check = 136;
+    // Devices which needs to enable leaf specific flag
+    // Juniper: b/319202763
+    bool enable_flowctrl_flag = 137;
 
     // Reserved field numbers and identifiers.
     reserved 84, 9, 28, 20, 90, 97, 55, 89, 19; 

diff --git a/proto/metadata_go_proto/metadata.pb.go b/proto/metadata_go_proto/metadata.pb.go