Add sleep 1 to prevent mininet packet storms in l2switch test 23/38823/2
authorLuis Gomez <ecelgp@gmail.com>
Thu, 12 May 2016 19:50:15 +0000 (12:50 -0700)
committerVratko Polák <vrpolak@cisco.com>
Fri, 13 May 2016 14:31:24 +0000 (14:31 +0000)
Forgot to add packet-storm prevent sleep in 2 test cases.

Change-Id: I86ed4fa4fa1d19f5c63606d9463b29bbd516700b
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
csit/suites/l2switch/Loop_Removal_OF13/010__loop_of_3.robot

index ce466788d03e645d5e764b0f5515ff03124af2ee..760f218663c92fcc08c6cc7cc22a34dfcbc9f55e 100644 (file)
@@ -57,6 +57,9 @@ Link Up
     Read Until    mininet>
     Wait Until Keyword Succeeds    10s    2s    Check For Specific Number Of Elements At URI    ${OPERATIONAL_NODES_API}    ${FORWARD}    4
     Wait Until Keyword Succeeds    10s    2s    Check For Specific Number Of Elements At URI    ${OPERATIONAL_NODES_API}    ${DISCARD}    2
+    # This sleep is needed because if the ping in the below WUKS is launched before the STP effectively removes the link,
+    # it produces a packet storm in mininet that makes the test unresponsive.
+    Sleep    1
     Wait Until Keyword Succeeds    10s    2s    Ping Works Good
 
 Remove Port
@@ -71,6 +74,9 @@ Add Port
     Read Until    mininet>
     Wait Until Keyword Succeeds    10s    2s    Check For Specific Number Of Elements At URI    ${OPERATIONAL_NODES_API}    ${FORWARD}    4
     Wait Until Keyword Succeeds    10s    2s    Check For Specific Number Of Elements At URI    ${OPERATIONAL_NODES_API}    ${DISCARD}    2
+    # This sleep is needed because if the ping in the below WUKS is launched before the STP effectively removes the link,
+    # it produces a packet storm in mininet that makes the test unresponsive.
+    Sleep    1
     Wait Until Keyword Succeeds    10s    2s    Ping Works Good
 
 *** Keywords ***