[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. |
| Comment by Faseela K [ 04/May/18 ] |
|
This has made my day dsuarezf I have been trying to run this locally with RunUntilFailure, but couldn't get the tests to fail, and it is passing. |