[OPNFLWPLUG-1061] add getFullyStarted() to OpenFlowPluginProviderImpl Created: 08/Jan/19 Updated: 29/Jan/19 Resolved: 29/Jan/19 |
|
| Status: | Resolved |
| Project: | OpenFlowPlugin |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Neon |
| Type: | Bug | Priority: | Medium |
| Reporter: | Michael Vorburger | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
add getFullyStarted() to OpenFlowPluginProviderImpl and fix the order of isOnlineFuture vs readyRunnable: For a test to be able to await getFullyStarted() and then check diagstatus, that (diagstatus) needs to be set BEFORE, otherwise there is a race condition. This has come up while writing the testDiagStatus() for the OpenFlowPluginWiringTest for opendaylight-simple. |
| Comments |
| Comment by Michael Vorburger [ 08/Jan/19 ] |
|
required by |
| Comment by Michael Vorburger [ 08/Jan/19 ] |
| Comment by Arunprakash D [ 29/Jan/19 ] |
|
vorburger, do we plan to cherry pick to stable/flourine? |
| Comment by Michael Vorburger [ 29/Jan/19 ] |
|
Nope, I don't ... the reason / background / motivation for this was to be able to writing the testDiagStatus() for the OpenFlowPluginWiringTest for opendaylight-simple, and that won't work on stable/flourine anyway for other reasons, so no need. Therefore closing this issue as Done. |