Merge "fix diagstatus NPE on showSvcStatus CLI"

OPNFLWPLUG-1010 Adopt mdsal changes proposed through weather item TSC-99

Change-Id: Ia13e428ccc93812da5d1b2d76473bd20659bc0b4
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Fix ClusterSingleton future type

The type changed from Void to ? extends Object, adjust for this
change.

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

fix diagstatus NPE on showSvcStatus CLI

Due to some bug, openflowplugin's service Descriptor was null, and in
such scenarios, when we execute "showSvcStatus" the CLI was throwing
some exception.  This needs fixes in two places, one in infrautils and
the second in openflowplugin. This one is for openflowplugin.

JIRA: INFRAUTILS-39
Change-Id: I965c55fefac11601c3231a43cfc0cc32ccaf57d8
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>

Remove deprecated Throwable.propagates

Change-Id: I23fa15bccb2f0a007b515e30c4197cb2f2ba06f0
Signed-off-by: Jozef Bacigal <jozef.bacigal@me.com>

OPNFLWPLUG-1008 : Make OpenflowPluginDiagStatusProvider ip address confiugrable

Currently if user checks diagstatus:showsvcstatus, then port status will
be checked for localhost. If user starts IP in different host and checks
for diagstatus:showsvcstatus, then it reads IP address from  switch
connection configuration xml file and checks port status for that IP
address. If no IP is specified in configuration file, then by default
localhost will be considered for port status check.

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

OPNFLWPLUG-1009 : Bind ports 6653/6633 on specified address

When user specifies ip address on config xml, the ports
should bind on the specified ip address.

Currently this piece of code was broken and this review
fixes the same.

Change-Id: I5def69bde14548682f4fa625acaefeb7feaa596f
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "must use full node-connector string when writing queue stats"

Fix IPv6 OXMHeader Mask issue for multi-layer

When programming an OVS flow to set an ipv6_src/dest as
an action field (i.e., set_field:2001::1->ipv6_src), the
current implementation is not checking the Mask and is
including the Mask (even for a full ipv6 address) in the
OXM header. Because of this, switch is rejecting the flow
with the following error.

"decode error: OFPBAC_BAD_SET_MASK. OXM header 0:32768:26:1:32
includes mask but masked OXMs are not allowed here"

This patch addresses this issue by checking the prefix and
including the prefix only if applicable.

Ref: https://git.opendaylight.org/gerrit/#/c/57694/

Change-Id: I75323601289712d98a67e1a3439f7bd41f88cba0
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

OPNFLWPLUG-990: Administrative Reconciliation alarm

The reconciliation can be triggerd in several ways. Once the reconciliation
is triggered on switches connected to a control node, the alarm will
be raised for each switch, which is implemented using JMX framework.
Once the reconciliation is complete for a particular switch, the alarm for
the particular switch is cleared.

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

must use full node-connector string when writing queue stats

OPNFLWPLUG-995
Similar to port statistics, when writing queue statistics into the
operational datastore, make sure to use the full md-sal node-connector
string. i.e. 'openflow:1:1' rather than '1'.

Change-Id: If9ae252c762ac26830539722481bcf11b58199e9
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>

Merge "OPNFLWPLUG-989: Counter for Administrative Reconciliation on multiple/all Nodes"

OPNFLWPLUG-989: Counter for Administrative Reconciliation on multiple/all Nodes

The implementation is done to find number of times admin reconcilation is
being triggered for each node.The counter value will be reseted during
controller restart and maintains the same during device disconnect.

Change-Id: I278a2875caef5f4cd7e9520ac48cd3f22c03724e
Signed-off-by: SujaT <suja.t@ericsson.com>

OPNFLWPLUG-992 : Device connection rate limiter implemenation

By default the device connection ratelimitter wll be disabled.

device-connection-rate-limit-per-min is the property available in
openflowplugin.cfg, which can be changed to control the rate at
which number of devices connecting the controller. This will be
useful in reducing load on the controller if there is more number
of switches connected at the time during cluster reboot.

Default value is set to 0, which disables device connection ratelimitter.

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

Merge "add default value to channel-outbound-queue-size"

Merge "Bump to odlparent 3.1.0 and yangtools 2.0.3"

add default value to channel-outbound-queue-size

We ran into an issue with puppet-opendaylight that if we
create the default-openflow-connection-config.xml without
channel-outbound-queue-size attribue opendaylight doesn't
listen to Openflow Port 6653.

This commit add default value to channel-outbound-queue-size
so that we won't need to spacify it in the
default-openflow-connection-config.xml

Change-Id: Ia13382973eab51369929a71c0864d62c240e01f7
Signed-off-by: Moshe Levi <moshele@mellanox.com>

Remove controller lldp dependency

Few pom files still has controller lldp dependency

Change-Id: I353cddb05a845bc354254d3a25261d0e6ac822ed
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "Logs are added to identify the flows and groups being added in the node for the corresponding transaction ID when it's pushed in the form of bundle messages going via bundle experimenter messages."

Merge "OPNFLWPLUG-986: Administrative Reconciliation for multiple/all Nodes"

Logs are added to identify the flows and groups being added in
the node for the corresponding transaction ID when it's pushed in
the form of bundle messages going via bundle experimenter messages.

Change-Id: I880a7f9e88535a6215912f9b17b63c9a9e9ce9de
Signed-off-by: eeiillu <suja.t@ericsson.com>

Merge "Add Karaf build profile"

Merge "OPNFLWPLUG-987: Apps not getting proper notification for the port status messages"

OPNFLWPLUG-986: Administrative Reconciliation for multiple/all Nodes

The mechanism that is currently used for switch reconciliation implies
either a switch or a controller restart.To avoid this force restarting
technique and to make the user more flexible administrative reconciliation
can be used.Here, the implementation is done to trigger switch reconciliation
operation between controller and switch through CLI command and
RESTCONF operation.

Change-Id: I11255685b12c73f1c7cb20a27b5176d6aad51199
Signed-off-by: eeiillu <suja.t@ericsson.com>

Bump to odlparent 3.1.0 and yangtools 2.0.3

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

Add Karaf build profile

Project local Karaf distros are handy for devs to test their work
however is unneeded by autorelease builds and should not be released
as part of the Simultaneous Release. Add a profile that is active by
default so that default behavior is unchanged however allows the
autorelease project to disable building this module.

Issue: RELENG-86
Change-Id: I135238264ca0732336276d65f58e04ac420bc62c
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "OPNFLWPLUG-993 ct_clear is not supported in Openflowplugin"

OPNFLWPLUG-987: Apps not getting proper notification for the port status messages

Whenever there is a port delete event, there will be a tx update
event followed by delete event. So, apps will know the node connector
delete is because of the actual port delete and not the node removal

When both the transactions is submitted at the same time, there is a
chance that update event might be missed as the submit future was
processed in the callback as asSync submit transaction.

Now, added explicit .get for port status message to do transaction as
syncSubmit.

Email discussion and CSIT results:
==================================
https://lists.opendaylight.org/pipermail/openflowplugin-dev/2018-March/008065.html

Change-Id: Ie6753d707c8bc6db1969c8ee6ab3bc862881ca57
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Add doc for bundle reconciliation usage

Add documentation for bundle reconciliation usage

Change-Id: Idfb1828015335bdd39df9796f48c6878e76aa6fc
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Update group add mod spec to reflect the correct details

Update the spec to reflect the latest changes

Change-Id: I87d92f1c75057fa6c0d4f5775e4036a7afd6efed
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

OPNFLWPLUG-993 ct_clear is not supported in Openflowplugin

Added ct_clear Action in nicira extension

Change-Id: I163ed99d5af8d3643749ac2a278f5c8170d7ae29
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Fix test identities

Identities are mapped to interfaces, not abstract classes. Fix
manual test identities.

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

Do not repackage guava

Guava is already provided by odl-guava-23, add a dependency on that.

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

Merge "Make TCs to assert on actual values and remove toString asserts"

OPNFLWPLUG-988: Service Montitor commands shows OPENFLOW as ERROR

Due to a NPE for a config file, showsvcstatus shows
openflow as error. Openflowjava southbound queue size
was missed in legacy switch config xml file.

Change-Id: Ic44645ece18c902b136cca94d9c0a12cf030d85e
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

OPNFLWPLUG-858/Bug 7826
Identified the root cause for issue TransactionCommitFailed for group during statistic collection.
Issue seems to be while the group being added to OperDS after adding it to config DS interms of cache,
it is completely made to be added only through statistic collection and not in any other transaction.
This way we can avooid the deletion of non existing group from OperDS after reading it from cache.

Change-Id: I8fe639e6102629fb903689114c617e76a61d68db
Signed-off-by: sunilkumarms <sunil.g.kumar@ericsson.com>

Make TCs to assert on actual values and remove toString asserts

Make assert on getValue insteadOf on toString values.

toString() method implementation change might break the ofp
TCs, to avoid these errors assert on actual values

Change-Id: I687ab07badca72f2f0489660ea5a8e0ebcaded07
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

OPNFLWPLUG-968: The channelOutboundQueueSize is made configurable:

The change is being made by adding a queue size parameter
to the already existing config file.

Change-Id: I94d467ecbeca8fbf3acf837e1b618f91867cf038
Signed-off-by: SujaT <suja.t@ericsson.com>

TCs in ofp failing as yang class toString method defintion changed

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

Due to patch in mdsal [1], yang generated toString is giving
different output and it is causing someof the TCs to fail in
openflowplugin.

Modified the comparision equivalent to the generated toString
output

Change-Id: Ibf7b4f629b296ffe59897f2206f8ab54fea5907b
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "OPNFLWPLUG-981: per-capability configuration for stats polling"

OPNFLWPLUG-981: per-capability configuration for stats polling

Implemented granular config knobs to enable or disable statistics
polling for each of the OpenFlow capabilities. Only if supported by
device and enabled by this configuration will the plugin poll the
statistics. By default all are enabled.

Change-Id: Ic0bc81f1f2efc51f96dc6ccd94e803b70ad14174
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>

OPNFLWPLUG-951: Adding equal role functionality

Change-Id: Ief56ad88f05ce8c89e04243d655d5f89d73a8d00
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@luminanetworks.com>

Merge "OPNFLWPLUG-983 Group and flow removal stats are not reported in order"

OPNFLWPLUG-984: always use ingress node for link discovery

Using the LLDP destination (packet-in ingress) guarantees that the
receiving controller is the owner of the device. Relying on the LLDP
source can cause a scenario where the LLDP packet never appears in the
entity-owner of the source device.

Change-Id: I8ad4171cdae395d260b754b95e5d9b75138fb79b
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>
(cherry picked from commit 36722d99c3a17bb23d4299a906591f11a9847f03)

OPNFLWPLUG-983 Group and flow removal stats are not reported in order

Change-Id: I6f66a40b0434d02fc32b3aa38f8e3834bad57005
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Enable findbugs in parent pom

Change-Id: Ib9c64da593973b23c0bf2d3bf499adc95e16df54
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in test-provider and drop-test-karaf

- Field only ever set to null
- int value cast to float and then passed to Math.round
- Possible null pointer dereference in method on exception path
- Non-virtual method call passes null for non-null parameter
- Comparison of String parameter using == or !=
- Should be a static inner class
- Private method is never called
- int value cast to float and then passed to Math.round
- Unread field
- Unread field: should this field be static?
- Unused field
- Dead store to local variable
- Private method is never called
- Method uses the same code for two switch clauses
- Write to static field from instance method
- Field not initialized in constructor but dereferenced without null check

Change-Id: Ib09380d626ab9162c4308810fae9db90522269cf
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in openflowplugin-impl

- Method ignores exceptional return value
- Check for sign of bitwise operation
- Consider using Locale parameterized version of invoked method
- Reliance on default encoding
- Synchronization performed on util.concurrent instance
- Unchecked/unconfirmed cast
- Unchecked/unconfirmed cast of return value from method
- Return value of method without side effect is ignored
- Dead store to local variable
- Method uses the same code for two branches
- Load of known null value
- Parameter must be non-null but is marked as nullable
- Field not initialized in constructor but dereferenced without null check

Change-Id: I92e00a5772038111d1ab8d9f53d52c43e3f4909d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in openflowplugin

- Comparison of String parameter using == or !=
- Comparator doesn't implement Serializable
- Field is a mutable collection
- Incorrect lazy initialization of static field
- Inefficient use of keySet iterator instead of entrySet iterator
- Private method is never called
- Unsigned right shift cast to short/byte
- Class is final but declares protected field
- Consider returning a zero length array rather than null

Change-Id: I714bd4005a33a772b0b76b0c0f89f28b8517cd06
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in extensions

- Dead store to local variable
- Unchecked/unconfirmed cast
- Field not initialized in constructor but dereferenced without null check
- Method ignores return value
- Private method is never called

Change-Id: I7a3fe1700439b456cb7dce6d84f63a9840fd68a7
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in samples

- Inconsistent synchronization
- Using notify() rather than notifyAll()
- Method ignores return value
- Read of unwritten field
- Field not initialized in constructor but dereferenced without null check

Change-Id: I12e8ae3ab25cc41ba05b99e9e4c85f2606a2814d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in liblldp

- Method call passes null for non-null parameter
- Class defines clone() but doesn't implement Cloneable
- Clone method may return null
- clone method does not call super.clone()
- equals() method does not check for null argument
- Reliance on default encoding
- May expose internal representation by returning reference to mutable object
- Field is a mutable collection which should be package protected - malicious code vulnerability
- Field should be package protected - malicious code vulnerability
- Inefficient use of keySet iterator instead of entrySet iterator
- Boxing/unboxing to parse a primitive - use parseXXX method
- Redundant nullcheck of value known to be non-null

Change-Id: I6d551d31d618a1adfc877ac59f29d02727201df6
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in applications

- Method ignores return value
- Consider using Locale parameterized version of invoked method
- Reliance on default encoding
- Write to static field from instance method
- Inefficient use of keySet iterator instead of entrySet iterator
- Dead store to local variable
- Consider returning a zero length array rather than null
- Consider using Locale parameterized version of invoked method
- Boxing/unboxing to parse a primitive - use parseXXX method
- Unread field
- Dead store to local variable
- Non-transient non-serializable instance field in serializable class
- Parameter must be non-null but is marked as nullable
- Field not initialized in constructor but dereferenced without null check
- Nullcheck of value previously dereferenced
- Redundant nullcheck of value known to be non-null

Change-Id: I1887f775f61fb8e3e6989113edf7faf4f9b06b34
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in openflowjava

- Invocation of toString on an array

- Check for sign of bitwise operation:
Example: ((event.detail & SWT.SELECTED) > 0)

Using bit arithmetic and then comparing with the greater than operator
can lead to unexpected results (of course depending on the value of
SWT.SELECTED). If SWT.SELECTED is a negative number, this is a candidate
for a bug. Even when SWT.SELECTED is not negative, it seems good practice
to use '!= 0' instead of '> 0'

- Reliance on default encoding - performs a byte to String (or String to byte)
conversion, and will assume that the default platform encoding is suitable.

- Primitive value is boxed then unboxed to perform primitive coercion

- Unread field

- Result of integer multiplication cast to long - possible overflow

- Dead store to local variable - assigns a value to a local variable, but
the value is not read or used in any subsequent instruction.

- Method uses the same code for two switch clauses

- Load of known null value

- Consider returning a zero length array rather than null

- Redundant nullcheck of value known to be non-null

- Field not initialized in constructor but dereferenced without null check

- Non-transient non-serializable instance field in serializable class

- Class implements same interface as superclass

Change-Id: I28b533c9f9805de539e35b729c5163fd22e75354
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Enable checkstyle enforcement in openflowplugin-parent

This covers all code sub-projects. Also removed checkstyle
configurations in all sub-projects.

Change-Id: Ib93f4abb41e97bc4c803cd54ad1c2ea56040d3db
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in test bundles

Change-Id: Id8e05a621e56dadea15252498bf04505f920441d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowplugin - part 5

Change-Id: I968d1e7931764b1e27e3aad40a3fd9094c24f270
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowplugin - part 4

Change-Id: Ib9e941db9d2f48b34f713a5618368aff68089e42
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowplugin - part 3

Change-Id: I0d8d7f5cc6c12299d67fbdacc13aafea7c086226
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowplugin - part 2

Change-Id: I83c41416dd64eba991ff95c4afc9e8a2b11e3a31
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowplugin - part 1

Change-Id: Ic66142803cdbc5fcb1e27d1ba8efc62aedceba04
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

use feature for inter-project dependency

Use feature to express inter-project dependency correctly instead of
bundle.

For details, refer to
https://lists.opendaylight.org/pipermail/release/2018-January/013604.html
https://lists.opendaylight.org/pipermail/release/2017-September/012356.html

This depends on below patch to export necessary bundle as feature.

https://git.opendaylight.org/gerrit/#/c/68318/

Change-Id: I38b4244d7c7622099285669d0ef190bd23c3dc90
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Merge "BUG-86: Fixed echo response processing"

Merge "OPNFLWPLUG-952: All links disappear from the topology"

Bump versions by x.(y+1).z for next dev cycle

Change-Id: Ib83c016ca4bdfc87da9ab847c874afe29a9a710a
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

OPNFLWPLUG-952: All links disappear from the topology

In cluster split situations LLDP Speaker can get into a bad state, with
incorrect number of node-connectors mapped locally. In some cases
multiple controllers can start sending packet outs. The solution here is
to use clustered DTCL to keep the node-connectors more consistent across
all instances of LLDPSpeaker and use EOS to determine who should
actually transmit the packet.

Change-Id: I36df734dff23688e8ace27b42f85291aa62ac391
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>

BUG-86: Fixed echo response processing

Change-Id: Ib83eb3463d101e872830d3985044c1154668baa1
Signed-off-by: Jalpa Modasiya <jmodasiy@luminanetworks.com>

Fix checkstyle violations in openflowplugin extensions

Change-Id: I6c2404ee9e0e30efab156a621bd7e641a78af972
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflowjava extensions

Change-Id: Ie4a01ec30e1feb054bfceb52a2598a2936264ace
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in samples

Change-Id: I5879e9d420c57c252d301ce7a5c296f8b5b914a6
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in applications

Change-Id: I1ac5ba94705ffba78d4765185ff6ade5f299a671
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "Fix checkstyle violations in liblldp"

export lldp as feature

Those bundles are used by genius and netvirt.
They should be consumable as feature, not bundle.

For detailed discussion, please refer to the followings
https://lists.opendaylight.org/pipermail/release/2018-January/013604.html
https://lists.opendaylight.org/pipermail/release/2017-September/012356.html

Change-Id: I034494a38c88c69c8d75731b73bab7b0f116db9d
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Fix checkstyle violations in liblldp

Change-Id: I1006a34064ca339a4280a255074eded82dbf5ce5
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

OPNFLWPLUG-974: Message deserialization failed

Description:

"Message deserialization failed...msgType: 36864 oxm_field: 6
experimenterID: null was not found - please verify that all needed
deserializers ale loaded correctly".

Observation:

The deserialization of this message failed due to the msg type of one of
the oxm fields (CtMark) was getting decoded wrongly.

This was due to the CtMarkCodec present in the packet(before the Metadata
oxm field) was decoding the field wrongly. The field doesnt have mask field
in it but still the mask it was read, ie, it was reading the next oxm field bytes too(Metadata)
thereby skipping the bytes from which the oxm Metadata is supposed to be read
and hence wrong data(skipped 4 bytes) is decoded in the next oxm field deserialization.

Fix:

The incorrect read of mask field in Ctmark codec has been removed now.

Change-Id: I79b6899686b05217df533e9ef365e9909dfc048d
Signed-off-by: gobinath <gobinath@ericsson.com>

Merge "OPNFLWPLUG-929 : Remove deprecated guava library"

Merge "OPNFLWPLUG-972: Point to openflowplugin liblldp"

Fix remaining checkstyle violations in openflowjava

Change-Id: I720977b96996a462946c5b6114a6b6b7dd1285bc
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 12

Many violations - more to follow

Change-Id: I909943b0b0b3e8974dd73422843238042b4bac1b
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 11

Many violations - more to follow

Change-Id: Iaa832ec597a543b1d963cd58307d7fed05f50720
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 10

Many violations - more to follow

Change-Id: I3b7ab07f56a13982f88df17805c3d82bbf14a062
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 9

Many violations - more to follow

Change-Id: Ide57e636e33d752cd9a6c912de071fd1493a7097
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 8

Many violations - more to follow

Change-Id: Ie7aac9d6bc6b1372ff425b89d83cf44b617f7751
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 7

Many violations - more to follow

Change-Id: I35f1ada078217cc89d03f2953829ac2274a53655
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 6

Many violations - more to follow

Change-Id: I18badf1a7b759dfbe63d240ea786c716d3135c97
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 5

Many violations - more to follow

Change-Id: I19ea3b92b66c0121946e88699af5e7c2742ea54d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 4

Many violations - more to follow

Change-Id: I9107de913adef9e19cdf7583b996598d4b9a63b0
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 3

Many violations - more to follow

Change-Id: I5cba0f1e5d8f61106db5d1461c205b80be585993
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 2

Many violations - more to follow

Change-Id: I360cd3cd431d1c05df1262df8debeb68f95ffac9
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle violations in openflow-protocol-impl - part 1

Many violations - more to follow

Change-Id: I3467223dd8ab3a9fe52906a2b029b0ab130b0b7d
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "Spec: OFPGC_ADD_OR_MOD support in openflowplugin"

Merge "Fix checkstyle violations in openflowjava-util"

Merge "Fix checkstyle violations in openflow-protocol-api"

Merge "Remove unused interface"

Merge "Remove dependency on sal-common-util"

Merge "Remove deprecated EOS services"

Remove dependency on sal-common-util

This dependency is no longer used, remove it along with use
of controller's odl-mdsal-common.

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

OPNFLWPLUG-929 : Remove deprecated guava library

use overload Futures.transform api

Change-Id: Iba64da122b31a022493a793f2e986bd91e530112
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "OPNFLWPLUG-969: ONF Extension is not installed due to which bundle based  resync is failing:"

OPNFLWPLUG-972: Point to openflowplugin liblldp

Change-Id: Ic029736e257e09825d35946287433bb41c6d61d3
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>