RT-5.4 TestPlan: Updated test Plan to capture intended behaviour of F…

…orwarding viable (openconfig#1642)

feature/interface/aggregate/ate_tests/aggregate_forwarding_viable_test/README.md

@@ -3,16 +3,31 @@
 ## Summary
 
 Ensure that forwarding-viable transition does not result in packet loss.
+Ensure that setting forwarding-viable=false impact **only** transmitting traffic on given port.
+Ensure that port set with forwarding-viable=false can receive all type of traffic and forward it normally (same as with forwarding-viable=true)
 
 ## Procedure
 
 *   Connect ATE port-1 to DUT port-1, and ATE ports 2 through 9 to DUT ports 2-9. Configure ATE and DUT ports 2-9 to be part of a LAG.
 
 *   For both Static LAG and LACP:
-    *   Make the following forwarding-viable transitions on a port within the LAG on the DUT.
-        *   Transition from forwarding-viable=true to forwarding-viable=false.
-        *   Transition from forwarding-viable=false to forwarding-viable=true.
-*   For each transition above, ensure that traffic is load-balanced across the remaining interfaces without packet loss
+    [TODO: https://github.com/openconfig/featureprofiles/issues/1553]
+    *   Run traffic bidirectionally between ATA port-1 and ATE port2-9;
+        *   ensure all ports has bidirectional traffic.
+        *   ensure that traffic is load-balanced across all port2-9
+        *   ensure ther is no packet losses in steady state (no congestion).
+    *   Make the forwarding-viable transitions from TRUE --> FALSE on a single port within the LAG on the DUT (one of ports 2-9).
+        *   ensure that above DUT's port is not sending any traffic to ATE (hence corresponding ATE port do not recive any traffic)
+        *   ensure that DUT load-balance traffic across remaing ports of LAG (7 out of port2-9)
+        *   ensure that above DUT's port is receiving  traffic from coresponding ATE port
+        *   ensure that there is no packet losses in steady state. (some very few packet could be lost right during transition)
+        *   ensure that DUT recives ~equal traffic on all ports of LAG (all port2-9)
+    *   Make the forwarding-viable transitions from FALSE --> TRUE on a single port within the LAG on the DUT (one of ports 2-9).
+        *   ensure that above DUT's port is sending traffic to ATE (hence corresponding ATE port do recive traffic)
+        *   ensure that DUT load-balance traffic across all ports of LAG (all port2-9)
+        *   ensure that above DUT's port is receiving  traffic from coresponding ATE port
+        *   ensure that there is no packet losses in steady state. (some very few packet could be lost right during transition)
+        *   ensure that DUT recives load-balanced traffic across all ports of LAG (all port2-9)
 
 ## Config Parameter coverage
 

feature/interface/aggregate/otg_tests/aggregate_forwarding_viable_test/README.md

@@ -3,16 +3,31 @@
 ## Summary
 
 Ensure that forwarding-viable transition does not result in packet loss.
+Ensure that setting forwarding-viable=false impact **only** transmitting traffic on given port.
+Ensure that port set with forwarding-viable=false can receive all type of traffic and forward it normally (same as with forwarding-viable=true)
 
 ## Procedure
 
 *   Connect ATE port-1 to DUT port-1, and ATE ports 2 through 9 to DUT ports 2-9. Configure ATE and DUT ports 2-9 to be part of a LAG.
 
 *   For both Static LAG and LACP:
-    *   Make the following forwarding-viable transitions on a port within the LAG on the DUT.
-        *   Transition from forwarding-viable=true to forwarding-viable=false.
-        *   Transition from forwarding-viable=false to forwarding-viable=true.
-*   For each transition above, ensure that traffic is load-balanced across the remaining interfaces without packet loss
+[TODO: https://github.com/openconfig/featureprofiles/issues/1553;]
+    *   Run traffic bidirectionally between ATA port-1 and ATE port2-9;
+        *   ensure all ports has bidirectional traffic.
+        *   ensure that traffic is load-balanced across all port2-9
+        *   ensure ther is no packet losses in steady state (no congestion).
+    *   Make the forwarding-viable transitions from TRUE --> FALSE on a single port within the LAG on the DUT (one of ports 2-9).
+        *   ensure that above DUT's port is not sending any traffic to ATE (hence corresponding ATE port do not recive any traffic)
+        *   ensure that DUT load-balance traffic across remaing ports of LAG (7 out of port2-9)
+        *   ensure that above DUT's port is receiving  traffic from coresponding ATE port
+        *   ensure that there is no packet losses in steady state. (some very few packet could be lost right during transition)
+        *   ensure that DUT recives ~equal traffic on all ports of LAG (all port2-9)
+    *   Make the forwarding-viable transitions from FALSE --> TRUE on a single port within the LAG on the DUT (one of ports 2-9).
+        *   ensure that above DUT's port is sending traffic to ATE (hence corresponding ATE port do recive traffic)
+        *   ensure that DUT load-balance traffic across all ports of LAG (all port2-9)
+        *   ensure that above DUT's port is receiving  traffic from coresponding ATE port
+        *   ensure that there is no packet losses in steady state. (some very few packet could be lost right during transition)
+        *   ensure that DUT recives load-balanced traffic across all ports of LAG (all port2-9)
 
 ## Config Parameter coverage