[NETVIRT-584] sporadic failures in the aclservice UT Created: 02/Apr/17  Updated: 06/Jul/17  Resolved: 06/Jul/17

Status: Resolved
Project: netvirt
Component/s: General
Affects Version/s: Carbon
Fix Version/s: None

Type: Bug
Reporter: Shlomi Alfasi Assignee: Slava Radune
Resolution: Cannot Reproduce Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

External issue ID: 8136

 Description   

acl tests are failing probably due to timing issue

 Comments   
Comment by Michael Vorburger [ 03/Apr/17 ]

likely related to use of Thread.sleep instead of correct await infra, possible introduced by https://git.opendaylight.org/gerrit/#/c/53002/ ... A concrete example of the problem occurring is here: https://jenkins.opendaylight.org/releng/job/netvirt-merge-carbon/org.opendaylight.netvirt$aclservice-impl/627/testReport/junit/org.opendaylight.netvirt.aclservice/LearnEgressAclServiceImplTest/addAcl__SinglePort/ : org.opendaylight.netvirt.aclservice.LearnEgressAclServiceImplTest.addAcl__SinglePort

java.lang.AssertionError: expected:<10> but was:<9>
(...) at org.junit.Assert.assertEquals(Assert.java:542)
at org.opendaylight.netvirt.aclservice.LearnEgressAclServiceImplTest.addAcl__SinglePort(LearnEgressAclServiceImplTest.java:161)

Comment by Michael Vorburger [ 03/Apr/17 ]

FTR: https://git.opendaylight.org/gerrit/#/c/54211/ adds @Ignore

Comment by Stephen Kitt [ 04/Apr/17 ]

I’m still seeing failures even with #54211 applied (e.g. https://git.opendaylight.org/gerrit/53909 which has been rebased on top of Yakir’s fix for #8135, which comes after Shlomi’s patch).

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