[GENIUS-120] InterfaceManagerConfigurationTest newl2vlanInterfaceTests is unstable Created: 10/Apr/18  Updated: 30/May/19

Status: Open
Project: genius
Component/s: None
Affects Version/s: None
Fix Version/s: Neon

Type: Bug Priority: Medium
Reporter: Michael Vorburger Assignee: Karthika Panneer
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

InterfaceManagerConfigurationTest newl2vlanInterfaceTests is unstable and occassional fails on the build servers, sometimes in merge and sometimes in verify jobs, recent examples:

https://jenkins.opendaylight.org/releng/job/genius-merge-fluorine/108/consoleFull#1624813939e5f83968-6a6e-4afa-aebf-7456048eb3ce (that was a merge job for https://git.opendaylight.org/gerrit/#/c/70445/)

https://jenkins.opendaylight.org/releng/job/genius-maven-verify-fluorine-mvn33-openjdk8/338/console (that was a verify job for https://git.opendaylight.org/gerrit/#/c/67468/)



 Comments   
Comment by Michael Vorburger [ 10/Apr/18 ]

Disabled in https://git.opendaylight.org/gerrit/70715 for now. k.faseela are you motivated to assign this to yourself, locally re-remove that Ignore, and using the RunUntilFailureRule see if we can figure out what's causing this? (NB that even if it does always pass locally with the RunUntilFailureRule, there could still be a problem - it may just only happen on slow build VMs, but still always pass locally; but doing that simple reproducing test is at least a first step...)

Comment by Faseela K [ 10/Apr/18 ]

https://git.opendaylight.org/gerrit/#/c/67468/ has migrated to AbstractListenerBaseClass which does not support awaitEventsConsumption(). I am not sure whether that is impacting the tests, as this listener is an important part of interface-manager tests.

Now anyways that we are going to ignore this, all patches can be merged without worrying about all these

And one fine day, when I enable it back, I have to fix all these.

Comment by David Suarez [ 03/May/18 ]

This change made the new listeners testable, so I think we could re-enable the test again.

[1] https://git.opendaylight.org/gerrit/#/c/71186/

Comment by Faseela K [ 04/May/18 ]

This has made my day dsuarezf  I think let us try to re-enable the tests and see how it goes.

I have been trying to run this locally with RunUntilFailure, but couldn't get the tests to fail, and it is passing.

Generated at Wed Feb 07 19:59:57 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.