Code Review / openflowplugin.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 9d7480b) | patch
add getFullyStarted() to OpenFlowPluginProviderImpl 98/78898/4
authorMichael Vorburger <vorburger@redhat.com>
Fri, 21 Dec 2018 00:46:15 +0000 (01:46 +0100)
committerMichael Vorburger <vorburger@redhat.com>
Tue, 8 Jan 2019 19:52:26 +0000 (19:52 +0000)
commita655d92afc526149467678d2f86e9730f961b70b
treeb02668baed34d88a6f585afef1c3c00cc5410115tree | snapshot
parent9d7480bac33a86eeb8cd4c16515c687ee975e608commit | diff
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.

JIRA: OPNFLWPLUG-1061
Change-Id: Ic9b5f9b49038fa75451e81ebaa1e6fbdbc41c683
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/TcpHandler.java diff | blob | history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/UdpHandler.java diff | blob | history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java diff | blob | history
OpenFlow Plugin