Merge "Cleanup JobEntry"

Merge "Simplify job NodeConfiguratorImpl dispatch"

Merge "Add task termination traces"

OPNFLWPLUG-644 : Documentation for openflowplugin event logging using
log4j

Change-Id: Ic85a0a2f212437af223a854ab6e7e162d9f46cd8
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Cleanup JobEntry

This is an implementation-private class, which is just a holder
DTO. The fields are not really mutable, hence make them final
and make sure we can communicate failures through it.

Change-Id: I5106329af0e193fb420ac0603033972bb56dea62
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify job NodeConfiguratorImpl dispatch

This simplifies the logic by eliminating an unneeded indirection.

Change-Id: Ia254d64d783ed4d465d70d2df4b9aa03bef27be4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add task termination traces

This adds trace messages to discern when execution of callbacks
is completed.

Change-Id: I61cee9098c9ef29e1e02b6e87edd06ade9178370
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Migrate to LoggingFutures"

Merge "Simplify DeviceContextImpl flow counting"

Merge "Simplify DeviceFlowRegistryImpl streaming"

Merge "Remove dead code from DeviceFlowRegistryImpl"

Merge "Update flow during arbitrator reconciliation should be handled as delete bundle message for original flow followed by add bundle message for updated flow."

Migrate to LoggingFutures

LoggingFutures provide more flexible method overrides, hence
allowing faster execution for known types (like ListenableFuture).

This migrates to use LoggingFutures instead of JdkFutures, also
eliminating unneeded temporary Future variables.

Change-Id: If2b1f99ffa6d09633706eb8c0ac323e116f3179e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify DeviceContextImpl flow counting

The code to count flows for debugging purposes is a rather arcane
stream. Simplify it by transforming some amount of code from
functional to imperative programming and taking advantage of
nonnullFoo() methods.

Change-Id: Ifa61e38b4a4e4ac786d598ba6199ed2d55304407
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify DeviceFlowRegistryImpl streaming

This patch takes advantage of Optional.ifPresent() and nonnullFoo()
methods to simplify the stream processing pipeline.

Change-Id: I9395a011fd7956d909b0b22d334aa9dcee601db6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove dead code from DeviceFlowRegistryImpl

newReadOnlyTransaction() and read() do not return nulls, hence
the code checking for them is dead. Remove it and refactor the
code to close the transaction immediately after issuing the read.

Change-Id: I9f02a19358c2bb0019b66a5778e489d52159499f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Declare reconciliation artifacts"

Merge "Reduce use of JdkFutureAdapters"

Merge "Cleanup use of Guava Function"

Merge "Remove use of Objects.{is,non}Null"

Merge "Take advantage of nonnullFoo()"

Update flow during arbitrator reconciliation should be handled as delete
bundle message for original flow followed by add bundle message for
updated flow.

This was not followed, so ended up in stale flows on computes after RBU.

Change-Id: Iff10dd6380a0636cf9f251ddd1e07c1bd88aad8f
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Remove use of StringBuffer

StringBuilder is the more modern replacement, use that.

Change-Id: If2a4cee7d41787b88800790830b09e4f6f01196d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of Objects.{is,non}Null

As noted in API documentation, these methods are to be used with
method handles. Replace their use with simple null checks.

Change-Id: Ifa02d42d5b38f62ab871be9768fffb1ea86d7463
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reduce use of JdkFutureAdapters

All of our infra is on ListenableFuture these days, there is no
point in squashing that to Future and then going back to
ListenableFuture.

Change-Id: I887a4bab29465c90202a66a352b7ffe1f63d6e6e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Declare reconciliation artifacts

This reduces the use of project.version and publishes reconciliation
framework to artifacts, so it can be consumed without references to
features.

Change-Id: I79ed1ff4909d8e15aaed5a7adf6d71d239f8ec5d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup use of Guava Function

We do not need to explicitly reference Function, but can use
java.util.function.Function and lambdas.

Change-Id: Ic2c84ab14fc5e014d8aaba09274ebb3a45fd649f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Take advantage of nonnullFoo()

Rather then having inline checks for nulls, take advantage of
built-in nonnullFull() method.

Change-Id: I20cc90de516ab8d335ad0a7353653218e5f47ae2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump mdsal to 4.0.2

This picks up latest fixes.

Change-Id: Ic8d7b12750d3335233905bfad5cf742d96a8ceac
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "OPNFLWPLUG-1027 : Topology manager writes link information everytime topology-lldp-discovery publish the linkAdded/linkRemove notificaiton"

OPNFLWPLUG-1056 : Default table flows missing

FRM registers for ownership callback and will get notified once master
is elected for a device. In some cases, Rolecontext is taking time for
ownership election but deviceContext is going ahead and writing the node
information to the oper inventory. So, apps which listen for node DTCL
will go ahead and push default table flows which might be dropped by FRM
as it has not yet got ownership callback.

The new implementation would be for devicecontext to wait for mastership
election to go through and then write the switch information to the oper
inventory. This will make sure FRM always has the mastership details
when it receives flow information.

Change-Id: I55d467aae68219adc1dfa6ad980bc27342d65191
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Merge "OPNFLWPLUG-644: Openflow Event logging using log4j"

OPNFLWPLUG-1027 : Topology manager writes link information everytime topology-lldp-discovery publish the linkAdded/linkRemove notificaiton

 Send new notifications only when the links are discovered
 and ignore already present links.

Change-Id: I5ec6ae4d2cd8d2c83eb6e918f12cf12927056633
JIRA: OPNFLWPLUG-1027
Signed-off-by: Gobinath <gobinath@ericsson.com>

OPNFLWPLUG-644: Openflow Event logging using log4j

Main events of openflowplugin can be logged using a separate common
logger, which will be using log4j logging framework. This can be
configured as a log appender in etc/org.ops4j.pax.logging file, through
which we can redirect main openflowplugin event logs to a separate file.
This can be helpful in debugging the problems with different phases of
device connection to the SDN controller.

The log appender can be configured as below:
========================================================================
log4j2.logger.ofp.name = OfEventLog
log4j2.logger.ofp.level = DEBUG
log4j2.logger.ofp.additivity = false
log4j2.logger.ofp.appenderRef.OfEventRollingFile.ref = OfEventRollingFile

log4j2.appender.ofp.type = RollingRandomAccessFile
log4j2.appender.ofp.name = OfEventRollingFile
log4j2.appender.ofp.fileName = \${karaf.data}/events/openflow/openflow.log
log4j2.appender.ofp.filePattern = \${karaf.data}/events/openflow/openflow.log.%i
log4j2.appender.ofp.append = true
log4j2.appender.ofp.layout.type = PatternLayout
log4j2.appender.ofp.layout.pattern = %d{ISO8601} | %m%n
log4j2.appender.ofp.policies.type = Policies
log4j2.appender.ofp.policies.size.type = SizeBasedTriggeringPolicy
log4j2.appender.ofp.policies.size.size = 10MB
log4j2.appender.ofp.strategy.type = DefaultRolloverStrategy
log4j2.appender.ofp.strategy.max = 10
========================================================================

Change-Id: I5e86e58fb4fcbd8048dae4fea92f2021bfdbdd04
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Bump mdsal to 4.0.1

This bumps mdsal to 4.0.1.

Change-Id: Id1c38ee68b29d240d8f7a1fc25851afa7bfec6be
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Delete ScenarioService"

Merge "Add IpConversionUtil.hasIpv4Prefix()"

Merge "Remove Optional.ofNullable() antipatterns"

Merge "Use ClassToInstanceMap instead of a HashMap"

Merge "Add NodeConfiguratorImpl enqueue trace"

Merge "OPNFLWPLUG-1062 Include additional LLDP fields in liblldp"

Delete ScenarioService

This isn’t used anywhere, if it’s not going to be then we might as
well delete it.

Change-Id: Iaec40bb9b96f8e8eb588370e3d45633af9596cc6
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Update docs version for Sodium

Change-Id: Id447d10f96ae7c59baf6c974ff3479d4cf27b2e8
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Autorelease sodium-mvn35-openjdk11 failed to build
openflowplugin-extension-api from openflowplugin

Change-Id: Ib0bccc0e2507cf0bd52b652ae990a3b7c640a9b1
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Bump mdsal to 4.0.0

This bumps mdsal to 4.0.0, adjusting for codegen changes.

Change-Id: If3d2e2c6ec1c70b95b3aae2105fe54de6c447000
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent to 5.0.0

This bumps odlparent to 5.0.0, adding JSR305 dependencies where
needed.

Useless annotations in methods specified by a contract
are removed, as SpotBugs reports them (rightfully) as errors,
namely NP_METHOD_PARAMETER_TIGHTENS_ANNOTATION, due to Guava
using checker.qual instead of JSR305.

Change-Id: I7646f5a01b9cf39f7eba316baa8d37c790630c29
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix spotbugs logging complaints

Logging empty or sign-only messages, as well squashing exceptions
draws complaints from newer SpotBugs, fix them up.

Change-Id: I33e9e23c2c347f269517d293b99c1a80ef37f5d6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use deprecated Futures methods

Futures require an executor, pass down directExecutor() to be
future-proof.

Change-Id: Ib68c29186ba3e57f4c59a4782f1604b17c4dae38
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

OPNFLWPLUG-1069 : Openflow service state is not marked as operational in showsvcstatus

Openflow service status depends on OPENFLOW_SERVER_6633 and
OPENFLOW_SERVER_6653 services. As OPENFLOW_SERVER_6653 marked along with
ip address, there was problem in deciding openflow service state.
Removing ip address from OPENFLOW_SERVER_6653 service name.

Change-Id: I4b650310ae8ef925a2f1aeb6a5de236d57433c3f
Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>

Add IpConversionUtil.hasIpv4Prefix()

This is a mirror method to the IPv6 case. The IPv6 version is
also optimized to prevent unnecessary reboxing via a shared internal
utility method.

JIRA: OPNFLWPLUG-1067
Change-Id: Ic32b95a52a197e5735f1740cbd7b2cb3530baa99
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove Objects.{is,non}Null abuse

These utility methods are primarily destined for use with method
handles, not for use in if statements and the like. They also have
more concise representation in bytecode -- through IFNONNULL.

Remove the abuse, resulting in shorter code and bytecode.

JIRA: OPNFLWPLUG-1066
Change-Id: Iff9073d27f65defcfb5e31e8fdea7d7c02c03d0c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Prevent Ipv4Address rewrap in OfToSalIpv4DstCase

We have an Ipv4Address on input, there is no point in unwrapping
and then rewrapping it -- just use it as is.

Change-Id: I271f0886e904077b3193d65407963e327779090b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix IpConversionUtil.extractIpv4AddressMask()

The algorithm here is wrong when prefix length is 0, as it
returns 255.255.255.255, where it should be returning 0.0.0.0.
This turns out to be an error coming from the fact we use a long
bitmask and incorrect masks to mask out the topmost long bits.

Use an int bitmask and switch to non-sign-extending shifts, which
fixes the problem.

JIRA: OPNFLWPLUG-1063
Change-Id: Idd42ba510a3a9d2032646f07b0bde8061ecd2637
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Pre-construct prefix masks

IPV4 can hold at most 33 different masks, which are being held
in immutable arrays. We can save significant amount of allocation
and parsing if we just pre-compute these quads.

JIRA: OPNFLWPLUG-1067
Change-Id: I8eb5507b9615373f0526e7bb2e8a1d7132634295
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize IpConversionUtil.extractIpv4AddressMask()

Rather than going through String.format() use simple pre-allocated
StringBuilder.

Furthermore, Ipv4Prefix is guaranteed to always contain a slash,
hence we can use a simple indexOf/substring instead of splitting
the string.

Finally, if the prefix is 32, we can reuse the DottedQuad we keep
around for full arbitrary mask.

JIRA: OPNFLWPLUG-1067
Change-Id: I99f036c86606142a9f844507789293cd9822692f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize IpConversionUtil.isArbitraryBitMask() string splitting

We do not need two intermediate lists to iterate over the result
of splitting. We also can pre-compile the splitting pattern.

JIRA: OPNFLWPLUG-1067
Change-Id: I2ff6b50d203ef90d8953364368cbbbe289d34935
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove duplicate IpConversionUtil.isArbitraryBitMask()

The ipv4 and ipv6 versions can be shared by passing an address
length argument, reducing code duplication.

JIRA: OPNFLWPLUG-1067
Change-Id: I11dcafa9823a05463164c343aa83f9f4826702a1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize IpConversionUtil.createArbitraryBitMask()

Refactor the method to use a constant DottedQuad for the no mask
case and clean up error logging.

JIRA: OPNFLWPLUG-1067
Change-Id: I377002d60d26d8a7c0c10e2a59de68186761326b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize splitter usage

Do not instantiate short-lived splitters, but reuse shared
instances. Also use splitToList() instead of rolling our own
conversion.

JIRA: OPNFLWPLUG-1067
Change-Id: I515a30bbb2b2cbfbd165a593915151cb52dd6474
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize IpConversionUtil.extractIpv4Address()

Rather than instantiating Ipv4Address ourselves, use IetfInetUtil
just as we do in the IPv6 case.

JIRA: OPNFLWPLUG-1067
Change-Id: Ie6a4b0ffe57adef2d884f1b0a3987b31f9c0795b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove Ipv{4,6}Address-based splitToParts() methods

These methods are marked as abuse, are inefficient and are actually
not used anywhere. Remove them.

JIRA: OPNFLWPLUG-1067
Change-Id: I58a75c96fa057faafadc27b459ae2412c7ae3f1d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove Optional.ofNullable() antipatterns

Use of Optional.ofNullable().{map,orElseGet,...} are an antipattern
which is inefficient (as it hides logic from the compiler) and
places burden on EA to prove the objects can actually not be allocated.

This patch eliminates the trivial uses, leaving some more complex
flows to be untangled later.

Change-Id: Id05292b125dcfac4f77d2ea0a70a01ef04a78a0c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use ClassToInstanceMap instead of a HashMap

ClassToInstanceMap provides a type-safe equivalent of what is
being coded up here.

Change-Id: I682a935ba2c255443b4fd4689d0c589bc0c2bac2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Remove redundant exception declarations"

Merge "Remove redundant type specifiers"

Remove redundant exception declarations

This allows callers to be simplified.

JIRA: OPNFLWPLUG-1045
Change-Id: I8ed5eecc20bb66c0b322ef3716612cad24bd8ff1
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Remove redundant type specifiers

This removes generic types and casts which aren’t necessary.

JIRA: OPNFLWPLUG-1044
Change-Id: I267045bb85b7c1c657fe9f9a2ac869d9914f3eb0
Signed-off-by: Stephen Kitt <skitt@redhat.com>

OPNFLWPLUG-1064: Openflow diagstatus: Merge all
the 3 openflow services into one "OPENFLOW"

Change-Id: I6d192ca72153f48892d8ee14c6e0d10f41c40dfc
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Add missing annotation dependencies

This ensures they are correctly flagged as optional.

Change-Id: I16931ddc763fe2150ebbd0f701ddf24a7df65748
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch to MD-SAL APIs

This mass-converts all components to use MD-SAL APIs.

Change-Id: Ied51c12836fc1fc91038557e66f02f4d9f0d6b0d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of internal Netty class

Java 8 offers ConcurrentHashMap.newKeySet(), which returns
a concurrent set. Use that instead of Netty's internal class.

Change-Id: I1be0dd7325aa8f3c9dc604afebe55f30221d37a4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

OPNFLWPLUG-1058: Changing INFO  to DEBUG

Change-Id: I61260b9deea227a99b927b5550bf54eeaa064598
Signed-off-by: Shweta Chaturvedi <shweta.chaturvedi@ericsson.com>

Bump versions by x.y.(z+1)

Change-Id: I2068a98cf1f178671f9b8c3809ffbee71b0bfa15
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

remove 2 old fixed Maven plugin versions for Java 11

In the hope that this fixes the problem seen on
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/autorelease-release-neon-mvn35-openjdk11/7/

Change-Id: Ie4bc450adbef1a69491e5a7e6134ac7de91f6000
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Do not override javadoc plugin version

We are currently at version 3.0.1, do not use an outdated version.

Change-Id: Ia8d3948bdf9b3a88708e0f1a8620f69d898d9dcf
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Rework bit-copying functions and re-enable tests"

Merge "Fix BitBufferHelper::toNumber()"

Merge "Clean up instance checks and casts"

Rework bit-copying functions and re-enable tests

This patch addresses the discrepancy between the insertBits() tests
and setBytes(), by making the source of the bits explicit (MSB or
LSB).

Change-Id: Ib88398b08730149dd4bb45382c84732dfbd6903b
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix BitBufferHelper::toNumber()

Static analysis suggested that toNumber() was incorrect (there was a
redundant write to value). Adding test cases to the UT showed that
this was the case; this patch adds the new test cases and fixes
toNumber(). It also switches assertTrue() to assertEquals() where
appropriate and cleans up a number of redundant initialisers.

Change-Id: Ia31bc91bd63f2f406dfc2d9194bf0ae4c4274055
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Clean up instance checks and casts

Change-Id: Ic343131694d71e5d775bd8519492da153541fb18
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Add javax.annotation-api dependencies

Java 11 no longer ships these annotations, hence they need to be
declared. The annotations are provided by the container, hence
we scope them as provided.

Change-Id: Idc66357f723eb62f8b7a8af3db533f11215fefbc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use scope=provided for karaf shell

This dependency is actively filtered by odlparent, but its transitive
dependencies are still leaking to our features.

Lower the scope to provided to get rid of those dependencies.

Change-Id: I4d9b30f4d94c15c1f27f74ca63c0defe430ec598
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add jaxb-api dependencies

Java9+ does not ship JAXB, add dependecies on jaxb-api to make
Java 11 build pass.

Change-Id: I8eca4865d8029a5b29c1abdca6ac08befa5f6aa2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "minor: clean up TcpConnectionInitializer / SwitchConnectionProviderImpl"

Merge "add getFullyStarted() to OpenFlowPluginProviderImpl"

minor: clean up TcpConnectionInitializer / SwitchConnectionProviderImpl

also make TcpConnectionInitializer getIsOnlineFuture() actually work;
even though this, as far as I can tell, was over engineered and does not
actually appear to be used, it still nicer to be correct as it helps
understand how OnlineProvider / ServerFacade appears to have been
intended.

Change-Id: Ide74d22a42960432b6114d666c7067f66803baf0
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bump to mdsal 3.0.6

Change-Id: I8c4ad63fd4a9f271ddb2318c3cf33afcfd9d539a
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Add NodeConfiguratorImpl enqueue trace

This adds traces about when a task is enqueued.

Change-Id: I250d2165d303daf1323fc98d3dd2c5f4c711b3d3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump mdsal to 3.0.5

This aligns with odlparent-4.0.9 and yangtools-2.1.8.

Change-Id: Id26aac37a504d62c7352d47b84c1e9a83df6e958
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump to odlparent 4.0.9

Change-Id: Ia744d7a9c4b1f6b5c7e3743b0fbf40228a452787
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Merge "mv log4j.xml from src/main => src/test"

Merge "Bump mdsal to 3.0.4"

Merge "Bump odlparent to 4.0.8"

Merge "Log the address and port when binding fails"

Bump mdsal to 3.0.4

This realigns odlparent/yangtools/mdsal and brings in the latest
improvements.

Change-Id: I8fa610d61cf5d82012b72153c10efe14ee8de753
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent to 4.0.8

Change-Id: Ia352d37507b22347944c13843c05dee2e8cc2819
Signed-off-by: Stephen Kitt <skitt@redhat.com>

OPNFLWPLUG-1062 Include additional LLDP fields in liblldp

Change-Id: Ibf3f34c77921fbe032198d8f390b834ced3e6c6a
Signed-off-by: priyankachopra <priyanka71188@gmail.com>

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>

Revert "Change log level for noisy disconnect log"

This reverts commit 00188709ae7cebe12da693f8dbe11108f940a4e3.

Change-Id: Ibd34cdc79d52ad74786c94f6f0cd25179c590970
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>