Merge "Add SwitchCertificate attributes in TLS failure notification"

Merge "Add INFO.yaml for openflowplugin"

Merge "Added JSON and XML payloads tabs with RFC 8040 URL"

Added JSON and XML payloads tabs with RFC 8040 URL

Signed-off-by: Dhiraj Sharma <dhiraj.8.sharma@gmail.com>
Change-Id: I4efd1b236dae9a93c02cad1c9e3a99600d45d3b4

Add SwitchCertificate attributes in TLS failure notification

JIRA : OPNFLWPLUG-1094

Change-Id: I00aa25b3dfc33d10333a43b8bc67a2515f8fc874
Signed-off-by: Harshini M <hm@luminanetworks.com>

Merge "Fix ExperimenterActionSerializerKey initializer use"

Merge "Make methods static"

OPNFLWPLUG-1093 : Exceptions Multiple entries with same key seen in openflowplugin

description: Post aluminium mri changes node statistics rpc response was not successful,
             as flowstatisticsmaplist list contain duplicate flow-Id keys,removed flowstatisticskey from Flowstatistics
Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I014e18630d1531df2f13e4d0c83defa183cadf0d
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Fix ExperimenterActionSerializerKey initializer use

Fix raw type use and convert tests to use Uint32 constants,
fixing migration warnings.

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

Make methods static

Eclipse points out these methods can be made static, clean up those
warnings. This very slightly improves performance as well as makes
it clear these methods are not touching object state.

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

Migrate IetfYangUtil.bytesFor() users

There are more types handled by IetfYangUtil, hence its naming is
evolving. Convert use of bytesFor() to to use macAddresBytes().

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

Remove use of Byte(byte) constructor

This constructor is deprecated for removal, do not use it.

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

Add INFO.yaml for openflowplugin

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

Openflow Event logging using log4j - addition/modification of few logs

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I9a4c0a74611eb2efcb359cac203959bdc4b8429e
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Correct ampersand escapes

Ampersands need to be escaped in XML. Since we are using XML highlight,
the examples failed to lex, leading to a rtd job failure.

Change-Id: I233e018b706c65354fd7a68f78f66505e2f34e30
Signed-off-by: Dhiraj Sharma <dhiraj.8.sharma@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix MRI feature references

OFP features are referencing a number of MRI features without using
ranged imports. Fix this up, improving our ability to upgrade
seamlessly.

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

Integrate controller-2.0.2

This bumps versions to the following:
- odlparent-7.0.3
- yangtools-5.0.3
- mdsal-6.0.2
- controller-2.0.2

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

Merge "All openflow nodes are not getting listed in the output of getallnodes  in controller nodes"

Integrate controller-2.0.1

This bumps versions to the following:
- odlparent-7.0.2
- yangtools-5.0.2
- mdsal-6.0.1
- controller-2.0.1

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

All openflow nodes are not getting listed in the output of getallnodes  in controller nodes

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: If64f441b211489828867ad9f2091260992ca7a5e
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

OPNFLWPLUG-1090: ConcurrentModificationException: null when connecting cbench switches

Issue:

CSIT job using cbench was frequently failing because some switches are
getting their initial connection rejected.

Observation and Fix:

On analyzing the logs it was found that the failure had been caused by
the sessionstats collection during initial handshake. "ConcurrentModification"
exception was thrown as multiple switch sessions were trying to modify
the SESSION_EVENTS map which reulted in the connection getting closed.

This has been fixed by making the SESSION_EVENTS threadsafe now.

Change-Id: I2d0ec7c00d7c893d6271942f5fdbe9815069d4d0
Signed-off-by: Gobinath <gobinath@ericsson.com>
(cherry picked from commit bd28cb92e0f6f3bca79a04a5e5a57c617b6cb552)

Set version to Aluminium

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I178492ba8affcb8eb74ee1e1e8ac5758a64b3727

Update MRI projects for Aluminium

This bumps versions to the following:
- odlparent-7.0.1
- yangtools-5.0.1
- mdsal-6.0.0
- controller-2.0.0

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

Merge "OPNFLWPLUG-1080: Nodes not populated in operational/opendaylight-inventory:nodes after Controller Isolation OPNFLWPLUG-1082: When switch changes owner, all cluster members delete switch inventory operational"

Fix checkstyle violations

Upgraded checkstyle is finding these, fix them up.

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

Remove invalid annotations

@Nullable on primitive 'int' or 'byte' is invalid, remove it.

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

OPNFLWPLUG-1080: Nodes not populated in operational/opendaylight-inventory:nodes after Controller Isolation
OPNFLWPLUG-1082: When switch changes owner, all cluster members delete switch inventory operational

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I6ceb3e71dcf86bd83fc9a3f03621502ac5144e95

Merge "OPNFLWPLUG-1087: ODL controller to provide view of openflow node reconciliation status"

MDSAL API migration changes

Desc: List of MdSAL API migration changes.

Signed-off-by: Somashekhar Javalagi <somashekhar.manohara.javalagi@ericsson.com>
Change-Id: I857c25b6bfcfe80160f735bac91746d113c844f3
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

OPNFLWPLUG-1087: ODL controller to provide view of openflow node reconciliation status

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: Ibf3e02fe4352a2bd559b838f8788ca18bc2ab83d
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Merge "Reduce use of deprecated constructors"

Use Magnesium controller

As part of controller transition to being MRI, downgrade the version
used to Magnesium release. This will be bumped to Aluminium proper
during the MRI window.

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

Reduce use of deprecated constructors

Integer-based constructors are deprecated, this adds a dent to
remove their usage.

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

OPNFLWPLUG-1083: Stats frozen after applying 2 sec delay in OF channel

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: Ie7fa92f2307ac5751f0c14133d582262f1c46f89

OPNFLWPLUG-1085: Switch-idle events weren't consumed by inbound handler during read time-out,
                 hence moved idle handler above inbound handler from end of channel pipeline

desc: Reverting the changes provided for OPNFLWPLUG-1039, by moving idle handler above the inbound handler
within channel pipeline, as the switchidle event message are fired after channel messages are consumed.

Signed-off-by: dheenadayalan b <dhinua@gmail.com>
Change-Id: I1c1a2306eb298ba72aad2480a8f34fa6affa9a28
Signed-off-by: dheenadayalan b <dhinua@gmail.com>

Merge "OPNFLWPLUG-1086: Reconciliation framework failure when starting cbench tool for the first time"

OPNFLWPLUG-1086: Reconciliation framework failure when starting cbench tool for the first time

Issue:

CSIT job using cbench had consistent failure with the error message
starting starting with a "closed connection ... exiting" message.
The karaf logs indicated the switches getting disconnected because of
"reconciliation framework" failure.

Observation and Fix:

On analyzing the logs it was found that the "flownodereconciliation"
service of the reconciliation framework was throwing an uncaught
exception (ConcurrentModificationException) which had resulted in
the reconciliation framework failure.

This has been fixed and logs have been enhanced to debug similar issues.

Change-Id: I3cbcea2da32c342c83846f2041d82e921d7acf08
Signed-off-by: Gobinath <gobinath@ericsson.com>
(cherry picked from commit 1fdc1a25f9703550c678b4cd83188de26ad9b19d)

Rework MatchEntrySerializer

MatchEntrySerializer has a filter, which is being applied before
main serialization, and that is the only way it is being used --
we check if the match is present and if it is we run serialize().

This leads to quite a bit of code duplication as well as inefficient
execution, MatchSerializer performs two invocations and each of them
performs at least one traversal of the match.

Change matchTypeCheck() to serializeIfPresent(), which means
MatchSerializer does not invoke two method. This also allows us to
optimize the serialization process in AbstractMatchEntrySerializer
and its subclasses by traversing the match only once. This rework
makes it obvious that MatchEntrySerializer should not be an
OFSerializer, at all, as the inherited methods just confuse the
workflow.

Resulting code structure removes quite a bit of duplicated code
from the individual EntrySerializer classes and lends itself to
sharing functionality through common subclasses -- reducing the
number of possible method implementations at various places, thus
allowing better devirtualization and inlining.

The second part deals with the logic of serializing the header,
as all specific serializers have a constant header, which we can
easily precompute and emit with a single ByteBuf.writeInt() call.
Since external serializers have had the option to dynamically emit
a different header, this capability is retained via HeaderWriter
helper which is attached to each serializer. Since the number of
serializers is limited, this additional object poses little memory
overhead and is partially offset by the reduction of bytecode coming
from the elimination of oxmField/oxmClass/valueLength getters.

Because all our serializers have a constant length, we only provide
one concrete implementation of HeaderWriter, which is
ConstantHeaderWriter. Unless an additional implementation is provided
externally, this means that all header serialization calls are
readily devirtualized via CHA, thus the cost of this indirection
is zero.

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

OPNFLWPLUG-1083: Stats frozen after applying 2 sec delay in OF channel

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I3a34ed327237ec2f765291cc45305683f561ae88

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

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

OPNFLWPLUG-1074: table stats not available after a switch flap

Problem: csit test cases failing due to empty table stats

Root Casue: First time when table stats are collected, they
are written into operational inventory. But consequently at
the time of writing flow stats to datastore, table stats
transaction was still not complete. So when fow stats are
written with empty table stats which are read from
operational inventory

Fix: Collecting flow stats first followed by table stats,
which prevents overwrite of table stats.

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

Merge "OPNFLWPLUG-1084 Device operational is not getting created if device reconciliation is not enabled"

Revert "All-dpns are not getting listed in the"

This reverts commit 566c4ac8fcd1feaa501955791790a247f3e69b07.

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

Revert "Fix for DPNs not connecting to controller"

This reverts commit 6b6281680872611d795261dbf232c0131235bd51.

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

Revert "CSC  to provide a view of DPN re-sync"

This reverts commit 674bb50091db4628a50c08606f8f8b9f3c8a5e55.

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

CSC  to provide a view of DPN re-sync
status from CLI

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

Fix for DPNs not connecting to controller

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

All-dpns are not getting listed in the
output of display all-dpns in cics

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

Optimize ReconcileUtil.safe*()

Node exposes nonnullTables() etc., which we can take advantage of
to simplify code.

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

Merge "Code refractor and logging improvements"

Merge "Code Refractor and logging improvements"

Merge "NETVIRT-1638 Modify yang modules such that it can be validated using pyang"

Code refractor and logging improvements

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

Code Refractor and logging improvements

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

OPNFLWPLUG-1084 Device operational is not getting created if
device reconciliation is not enabled

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

NETVIRT-1638 Modify yang modules such that it can be validated using pyang

Fixing pyang warning and errors.

Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
Change-Id: Ic66aa4a4394b5b6132b4027e044da5b165e2912e

Moving threadPool to a common place.

It was creating one threadPool for each connection
and failed to close it, now using common thread pool
for all the incoming dpn connections on 6653.

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

Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9

Pick up latest fixes and upgrades.

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

Bump odlparent/yangtools/mdsal to 6.0.3/4.0.3/5.0.6

Pick up the latest upgrades and fixes.

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

Merge "Failed to cancel service reconciliation, When controller become slave."

Error log correction

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

Avoid holding netty threads during resync callbacks

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

Merge "Grep not working for Karaf commands"

Read action throwing NPE

Issue Description:
When process alien action throwing NullPointerException:
java.lang.NullPointerException: null
        at org.opendaylight.openflowplugin.openflow.md.core.extension.ActionExtensionHelper.processAlienAction(ActionExtensionHelper.java:46)
        at org.opendaylight.openflowplugin.impl.protocol.deserialization.util.ActionUtil.readAction(ActionUtil.java:66)

Signed-off-by: Ning Zhang <zhangninglc@inspur.com>
Change-Id: I4008eef616bb08de9333442a4f21c3f21b12c65e

Merge "OPNFLWPLUG-1078: Notify device TLS authentication failure messages"

Merge "OPNFLWPLUG-1075: Making Device Oper transactions atomic"

OPNFLWPLUG-1075: Making Device Oper transactions atomic

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I7b2ed09633367e01229c7ea83063525c688821e8

OPNFLWPLUG-1078: Notify device TLS authentication failure messages

Signed-off-by: VenkataSatya Jonnadula <rsankar@luminanetworks.com>
Change-Id: I2d6acae15e94676b23764c62f4094123014e0093

Grep not working for Karaf commands

SupressWarnings is used to suppress checkstyle warnings thrown upon
using System.out

This covers the changes for "southbound-cli" in openflowplugin

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

Refactoring shownode cli

Grep was not working for shownode command

SupressWarnings is used to suppress checkstyle warnings thrown upon
using System.out

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

Refactoring shownode and getallnodes cli

Changing OFNODE to NodeId in shownode cli
and removing extra line from getallnodes cli

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

Migrate MockitoJUnitRunner

This migrates from org.mockito.runners.MockitoJUnitRunner to
org.mockito.junit.MockitoJUnitRunner.

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

Merge "OPNFLWPLUG-1049 : Device connection hold time to maintain delay between consecutive connections to the controller"

Defaults flows missing due to mastership not available

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

Getallnodes should read from cache instead of DS

Problem
=======
 Get All DPNs query takes long time to respond, as it reads from database.

Solution
========
Fix is to provide cache based response instead of Data store read.

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

OPNFLWPLUG-1036 OFP deleting topology-netconf operational data

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

OPNFLWPLUG-1049 : Device connection hold time to maintain delay between
consecutive connections to the controller

Device connection hold time is the least time delay a device has to
mantain between its consecutive connection attempts. If time delay
between the previous connection and the current connection is within
device connection hold time, the device will not be allowed to connect
to the controller. Default value of the device connection hold time is
0 second.

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

Merge "Use String(byte[], Charset)"

Merge "Apply some Objects.requireNonNull() modernization"

Failed to cancel service reconciliation, When controller become slave.

1.An exception may be thrown when the previous service reconciliation is cancelled,
causing the subsequent service to fail to cancel.
2.Do not use the toString() method when printing logs

Change-Id: I14f22a4b637890abb530140ba3666fda2181e24a
Signed-off-by: Ning Zhang <zhangninglc@inspur.com>

Merge changes from topic "odlparent-6.0.1"

* changes:
  Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4
  Fix meter-id overlap

Merge "Optimize ArbitratorReconciliationManagerImpl"

Optimize ArbitratorReconciliationManagerImpl

Use constant objects more rather than building them over and over
again. Also use ListeningExecutorService to dispatch tasks, leading
to simpler code.

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

Do not use toString() in logging arguments

Logging implementation will call toString() when it is needed,
doing it explicitly just wastes resources when logging is not
enabled -- which is typically the case for debugs.

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

Use String(byte[], Charset)

This is pointed out by modernizer and is more efficient.

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

Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4

This picks up latest upgrades and fixes.

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

Fix meter-id overlap

Adding meter-id leaf in two cases is not valid YANG, fix that up.

Change-Id: Icbdf91015d28061548bac3aa0f1f94e1668aaa91
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit d9f901335a6fd2ff660545d4e85497fd64f7d533)

Apply some Objects.requireNonNull() modernization

This cleans up moderinization warnings reported for
 openflow-protocol-{api,impl} and simple-client.

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

Merge "OPNFLWPLUG-1077 : Bundle resync is not able to push groups to switch"

Merge "Eliminate {infra,service}utils.version"

OPNFLWPLUG-1077 : Bundle resync is not able to push groups to switch

As buckets are put into the SingletonImmutableList, sort on
SingletonImmutableList is not supported. yangtools is converting
yang list into SingletonImmutableList. Copying same into
ArrayList and sorting same.

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

Merge "Remove nagasena dependency declaration"

Merge "Make applications/pom.xml an aggregator"

Merge "Turn libraries/pom.xml into an aggregator"

Merge "Turn samples/pom.xml into an aggregator"

Merge "Turn extension/pom.xml into an aggregator"

Merge "Make model/pom.xml inherit from odlparent-lite"

Eliminate {infra,service}utils.version

Rather than going through properties, use foo-artifacts to get
proper version declarations into dependencyManagement. This cleans
up version management a bit.

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

Remove nagasena dependency declaration

This dependency was used by netconf a long time ago, we no longer
use it (and properly declare libraries). Remove the declaration.

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

Make applications/pom.xml an aggregator

This really is just an aggregator, make sure it inherits from
odlparent-lite and adjust its users to inherit from ofp-parent.
Shaves ~29 seconds from build time.

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