[VTN-19] integration test is unstable. Created: 13/May/14  Updated: 03/May/18  Resolved: 14/May/14

Status: Resolved
Project: vtn
Component/s: VTN Manager
Affects Version/s: unspecified
Fix Version/s: None

Type: Bug
Reporter: Shigeru Yasuda Assignee: Unassigned
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: Linux
Platform: PC


External issue ID: 1000

 Description   

manager.integrationtest was failed.

java.lang.AssertionError: null
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertTrue(Assert.java:52)
at org.opendaylight.vtn.manager.internal.VTNManagerIT.testIListenDataPacket(VTNManagerIT.java:4605)

This failure was caused by a conjunction of 4 bugs:

1. Illegal initial value of the count down latch.
This test removes flow entries installed by previous test.
So "ethers.size() * 2" must be used as the initial value of the count down
latch to wait for node0 flows. (VTNManagerIT.java: line 4518)

2. Too short timeout to wait for flow entries.
Current timeout value (2 seconds) may not be enough to wait for flow entries
to be installed if the build host is busy.

3. Need to ensure that the shortest path graph is updated.
This test needs to ensure that an edge from node1 to node0 is available
before sending packet.

4. A map which keeps installed SAL flow must be synchronized.
manager.integrationtest uses a mock-up of IPluginInFlowProgrammerService
in order to keep installed SAL flows. It needs to be synchronized because
SAL flows may be modified by multiple threads concurrently.



 Comments   
Comment by Shigeru Yasuda [ 13/May/14 ]

A patch was pushed.

https://git.opendaylight.org/gerrit/6937

Comment by Shigeru Yasuda [ 14/May/14 ]

This patch was merged to stable/hydrogen branch.

https://git.opendaylight.org/gerrit/6983

Generated at Wed Feb 07 20:47:47 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.