Use odlparent checkstyle

Patch set 2: Remove ovsdb_checks.xml from parent pom
Patch set 3: Remove ovsdb_checks.xml from directory
Patch set 4: Add hwvtepsouthbound

Change-Id: Iccc680728e19967f9977453bb75c9c64b1a53e0c
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Fix sonar integration test reporting"

Fix sonar integration test reporting

Change-Id: Icb238c6442a8d15ae144d6e9abbaabf9fd011570
Signed-off-by: Sam Hague <shague@redhat.com>

Added postman collection and postman environment
files for Southbound plugin to test in Single and 3 Node cluster environment

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

Merge topic/master/clustering to master branch
Squashed commit of the following:

commit e9e3a762ee75844b930d8251f44d2718a3358445
Merge: 2adbc25 83cfd5b
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Fri Oct 2 19:54:23 2015 +0530

    Merge branch 'master' into topic/master/clustering

    Change-Id: I3c26c3f4095b78d869cae271b01fa1ab2609cf89
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 2adbc25b474d46f394e8f956c818383425570600
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Fri Oct 2 07:22:44 2015 +0530

    Addressed following issuses:
    1) Fixed entity unregisteration
    2) Implemented logic for southbound provider to participate
    in entityOwnership service, Southbound plugin instance as an entity
    This will avoid all the instances to do initiatize the operational
    and config topology, only master instance (who got ownership) will do that.
    3) Properly closing listener registration for OvsdbConnectionManager listerner.

    Change-Id: I73693b4faa8a6409c81c33647836516002b4b2fd
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 193f145796924c051b36c61974df4253e68efb7e
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Thu Oct 1 16:54:41 2015 +0530

    Addressed following issues:
    1) Handled ownership change notifications for the device that is not registered by the instance.
    2) If there is no owner of the device hasOwner=false, then clean up the operational data store,
    3) For switch initiated connections, generate iid by fetching the openvswtich table row from the device.

    Change-Id: I03158a616e7c020ce8f851b8251e4162f38ba622
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit c26d79561555c23f4aec894418537cf059a1f85a
Merge: 740d21a 7848a77
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Thu Oct 1 14:42:00 2015 -0400

    Merge branch 'master' into topic/master/clustering

    Change-Id: I191e71030d6523a3d7a11765010cc1b81b9405f3
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit 740d21afacb5a652a3a1d78ceff2c0d998492add
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Wed Sep 30 02:15:26 2015 +0530

    Fixed iid issue for switch initiated connections
    and refactored the code a bit

    Patch 2:Fixed unit tests.
    Conflicts:
     southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/OvsdbConnectionInstance.java
     southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/OvsdbConnectionManager.java

    Change-Id: I9f8adfba4a4f866d423503b24f400c40bd43131b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 9079eb260e123c00625dc13e63a6f55d9bc42aea
Merge: b203967 0179232
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Tue Sep 29 12:58:54 2015 -0400

    Merge branch 'master' into topic/master/clustering

    Change-Id: I02c771238cd7447dbaa326516467259516868aa6
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit b203967504c1de8814c32e7c7c8015ebb749e154
Author: Sharad Mishra <sharad.d.mishra@intel.com>
Date:   Mon Sep 28 06:19:38 2015 -0700

    OVSDB cluster device ownership

    Only allow device owner to update device.
    Patch 2: Fixed unit test

    Change-Id: Id942a2ba16a83b19807e135a9b67f8a7eb7da003
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 218785ea03b74287de86395d1e57471ce02fe5e4
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Wed Sep 23 09:55:06 2015 -0400

    Use EntityOwnershipService to get ownership on the device.
    If ownership is granted then *this* instance of southbound
    plugin can perform actions to configure the device, rest
    two southbound plugin instanceses in the cluster will just
    hole the connection with them and won't do any operation
    of configuring the device or reading from the device.
    For controller initiated connections, all the southbound
    plugin instance will attempt to connect to the device and
    contest for ownership of the device and selected owner of
    the device will do CRUD operations. For switch initiated
    connection, only instance of southbound plugin that received
    connection from switch will contest for ownership.

    Change-Id: I349699ae0630e12bdbe18b6bd2588a651edd1f81
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit da209a4aad67400a3c938329fb393ec6f7e2439e
Merge: ec7827c 11c0865
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Sep 29 00:44:50 2015 +0530

    Merge branch 'master' into topic/master/clustering

    Change-Id: I44ac4d66f731c0c0f56d746c563c6210eeceb978
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit ec7827c688340fa85c42942ba6559201da07286a
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Mon Sep 21 17:57:02 2015 -0400

    Add hasDeviceOwnership flag to OvsdbConnectionInstance

    Patch set 4: fix ut

    Change-Id: Id7da9071374938c16b086b0d67059aec0a7cc3a0
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit 3cc32fd3d007b4c91dfbadee3920b9ae61a9283c
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Fri Sep 25 17:02:13 2015 -0400

    Set bridge controller(s) based on ovsdb node manager(s)

    Change-Id: I10bcc04aed78f874cbb8a7d9a82e59cf1b80ec82
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit 628749e43002c4aa9fc8bfdf7c827cba0532eb6b
Merge: 3d7ce06 360d519
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Sat Sep 26 01:27:28 2015 +0530

    Merge branch 'master' into topic/master/clustering

    Change-Id: I20e84a6971466357c6819c7dba5d51809f9b2878
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 3d7ce0660d25080dd1a5e3927eda6d071e085d12
Merge: 8a90856 708c8c3
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Fri Sep 25 06:07:23 2015 +0530

    Merge branch 'master' into topic/master/clustering

    Change-Id: I35038c6d7b2150d2efc9692a02d5e94bd97a8dba
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 8a908569f4e8e19e7f03f08f97d0704f9ca46cc9
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Sep 22 01:52:42 2015 +0530

    Enable remote notification for southbound plugin

    (cherry picked from commit 01f0737be6abacaf56721731f6a19dcb13e291f0)

    Change-Id: Ida036244f3861e4d07437b49179bbf2851a212a5
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 7747c272e6ad5f52ebcae97fd14e05762bc8f790
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Sep 22 00:05:49 2015 +0530

    Wire entity ownership service to OVSDB southbound plugin

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

Merge "Use the memorized service reference"

Merge "UT for SouthboundUtil.java"

Merge "Bug 4277 - Deleted Network flow entries retained in br-int, if network associated and disassociated from virutalrouter interface"

Future-proof OvsdbTypesIdResolver

Instead of implementing TypeIdResolver directly, the recommended
approach is to extend TypeIdResolverBase; this ensures that upgrading
Jackson won't cause sudden build failures because of methods added to
TypeIdResolver (TypeIdResolverBase provides an appropriate default
implementation).

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

Bug 4277 - Deleted Network flow entries retained in br-int, if network associated and disassociated from virutalrouter interface

        Remove the mac cache after deleting the DHCP neutron port l3 flows

Patch set 2: Move all other caches updated in delete into cleanupRouterCache()

Change-Id: I0c29ba4976ddfd484415d851ad5a2df97d61b19a
Signed-off-by: ravi_sabapathy <ravi_sabapathy@dell.com>

UT for SouthboundUtil.java

Change-Id: I87e3577ab2f262ec7f50b6c185a41f49b1e607ed
Signed-off-by: rpujar <rpujar@inocybe.com>

Merge "Speed up SouthboundIT through multi-threading test cases"

Merge "Add yang for hwvtep"

Use the memorized service reference

Currently when dispatching "event handler added" events we retrieve
the appropriate service reference from the bundle context, but this
seems to cause problems when multiple services are registered for the
same interfaces. Since we know the service reference we got when
registering the service involved, we can just pass that.

As a result, ConfigInterface::setDependencies() doesn't need a
BundleContext at all, and since we're no longer supposed to pass that
around, we can just remove it altogether.

Some more debugging information should help track down other issues of
this type.

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

Less restriction on shared networks: it can be allowed on externals

Change-Id: I0a66758731eb35a73623c6f663792b9dffbfec08
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Added model for QoS and Queue."

Use common profiles among ovsdb it bundles

Change-Id: I2636a15eb821e1be7edbebcdeef9d94cb0d783be
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Remove unused integrationtest"

Remove unused integrationtest

Also add missing dependencies in library

Change-Id: I120a20ffb76f8d18bb54b1b72a69e752922974e4
Signed-off-by: Sam Hague <shague@redhat.com>

Added model for QoS and Queue.

Change-Id: I0d254ea9648dee2298a62523d6a10b64dcc88297
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>

Merge "Added Security Rule for Custom ICMP"

Merge "Remove unused northbound"

Merge "Controller initiated connections always connect to one manager listening in ptcp: mode. In clustered mode, all the three controller will connect to the device, and in case any of the connection will flip that instance will remove OvsdbNode from operational data store because it will see only one manager in manager list. But because switch is listening in passive mode, there can be multiple connection to same manager. To handle this scenario, this patch adds number_of_connection attribute to OvsdbNodeAugmentation to maintain the number of active connection to specific manager. In case of switch initiated connection this value is going to be 1 for each manager, but in case switch listening in passive mode, this value will represent active connections (extraced from Manager tables status column n_connections attributes)."

Merge "Bug 3989 -- Fix the original patch. Original patch can break the functioning if there are more then one data change and the first data change is OvsdbNodeAugmentation for which connection already exist. It will return out of OnDataChanged() when it see that connection already exist, without processing rest of the data"

Merge "Move OvsdbClient it under library-it"

Bug 3989 -- Fix the original patch. Original patch can break
the functioning if there are more then one data change and
the first data change is OvsdbNodeAugmentation for which connection
already exist. It will return out of OnDataChanged() when it see
that connection already exist, without processing rest of the
data

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

Move OvsdbClient it under library-it

A following commit will remove the integrationtest dir. With this commit here there
are no longer any tests in integrationtest. They have all been moved into the respective
bundles.

Change-Id: I40e7dccd6e5ee7ea362d75a422610fbdfe5e55cc
Signed-off-by: Sam Hague <shague@redhat.com>

Controller initiated connections always connect to one manager
listening in ptcp: mode. In clustered mode, all the three controller
will connect to the device, and in case any of the connection will flip
that instance will remove OvsdbNode from operational data store because
it will see only one manager in manager list. But because switch is
listening in passive mode, there can be multiple connection to same manager.
To handle this scenario, this patch adds number_of_connection attribute
to OvsdbNodeAugmentation to maintain the number of active connection to
specific manager. In case of switch initiated connection this value is
going to be 1 for each manager, but in case switch listening in passive
mode, this value will represent active connections (extraced from
Manager tables status column n_connections attributes).

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

Remove unused northbound

Change-Id: I8ac5ae92315d2558a7563a35bdd67c0d816ccc68
Signed-off-by: Sam Hague <shague@redhat.com>

Fix ovsdb node update in clustered environment
In cluster environment same switch can be connected to multiple
controller instance running southbound plugin. In the existing implementation
if device disconnect from any of the controller instances OVSDB-SB remove
OvsdbNode from operational data store. In cluster environment that will
break contract with customer, given that device is still connected to
atleast one of the two controller instances running. This patch modifying the
behavior where controller will see if its the only connected controller,
then it will delete the data from operational data store, else it will skip
any clean up from operational data store, because other instances will anyway
update about the disconnected controller manager entry in the managers list.

Change-Id: Iaa043f6632eb2de660b623604e8f8e0938dbc4d5
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 6051e15c6f929c08f152417b1f19373b10e93647)

Merge "Remove unused mdsal-plugin-adapter code"

Speed up SouthboundIT through multi-threading test cases

This speeds up SouthboundIT without losing coverage through the use of parallel
test cases.  On my VM,ExecutorSer this reduced test time from ~12 to ~4.5 minutes.

Aggregates the testing work for certain test cases into corresponding
Runnables.  For each set of test input data, a Runnable is spawned and
submitted to a fixed thread pool.  The thread pool size may be altered
through the NUM_THREADS constant.  The default is 4 since raising this
number causes issues with MD-SAL.

These test cases can be multi-threaded because they each utilize a distinct
bridge.  In other words, each thread is manipulating a different bridge in the
operating system.

Change-Id: Icd8ca69171e996fbf4000ef632c9446bb6c89ed6
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Remove unused mdsal-plugin-adapter code

Change-Id: I161b63c1c5b4cff39388d8342052920e245e3796
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Mdsal utils bundle"

Merge "Initial pass at netvirt model"

Mdsal utils bundle

Patch set 2: Renamed to MdsalUtils and matched the strucure of
the other bundles in the utils dir.

This commit adds a Mdsal utils bundle that has all the mdsal methods to read,
write, delete and merge to the datastore. This is so that other bundles can
reuse the same methods.

Change-Id: Ida83ed957f74155224b007b54e3e80bfb1e29a71
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Added UT for OvsdbDataChangeListener.java"

Added UT for OvsdbDataChangeListener.java

Also added code to test new method in OvsdbConnectionManager.java

Change-Id: I75faec2f20305c9ac806087042136d09d1048d49
Signed-off-by: rpujar <rpujar@inocybe.com>

Initial pass at netvirt model

Patch set 2: Renamed netvirtmodel to net-virt-sfc.
Patch set 3: Add net-virt-sfc to the root pom so it is built

Change-Id: Ia8b21b538e38463e860148abb4bf055ae3020761
Signed-off-by: Sam Hague <shague@redhat.com>

Clean up dependencies and features

Remove more plugin-related code.
Correct a couple of netconf dependencies.
Remove obsolete dependencies on controller features which are no
longer provided upstream.

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

Add YangInstanceIdentifer to InstanceIdentifier converter method

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

Fixed unit test testDisconnect in OvsdbConnectionmanagerTest

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

Merge "hwvtepsb: Fixed restconf migration build issues"

hwvtepsb: Fixed restconf migration build issues

restconf migration from controller to netconf was done while
hwvtep bundle creation was in progress, so those were not done
for hwvtepsouthbound which is causing build failure.

Change-Id: Ib32c591e0dd197730f1bc50d7e92b38018c0fd7f
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Merge "Unit Test for ovsdb southbound OvsdbConnectionManager.java"

Merge "Added manager list to Ovsdb Node. For application southbound plugin only give a device level view and not a connection level view. To make net-virt cluster aware, it needs to the managers of the ovsdb server, so that when it creates ovs bridges, it can connect them to the same controllers through OF plugin"

Added manager list to Ovsdb Node.
For application southbound plugin only give a device level view and not a connection level view.
To make net-virt cluster aware, it needs to the managers of the ovsdb server, so that
when it creates ovs bridges, it can connect them to the same controllers through OF plugin

Tests:
ovs-vsctl set-manager tcp:ip:6640
ovs-vsctl set-manager tcp:ip:6640 tcp:ip2:6640
ovs-vsctl set-manager tcp:ip:6640
ovs-vsctl del-manager

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

Removing the old unused plugin and ovsdb-plugin-compatibility-layer
OVSDB stopped using these module in Lithium release. These modules were kept for VTN project, because they didn't had time to migrate
VTN component that relies on these module to MD-SAL.
We will merge this patch after one month on 19th Sep.

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

Fixed binding-parent groupId. It was causing build failure

Patch2: Fix library module to use odl-mdsal-models feature rather then deprecated odl-yangtools-models
Patch3: Fix hwvtep southbound module to use odl-mdsal-models feature as well. Also disable hwvtepsouthbound-karaf
 and hwvtepsouthbound-it features. They are causing the build break.

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

Merge "Revert "Use the library config bundle in southbound""

Revert "Use the library config bundle in southbound"

This reverts commit c031f4be42abdbee58417a0c0af3ea14f74b2e95, which fails IT currently.

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

Merge "Use the library config bundle in southbound"

Unit Test for ovsdb southbound OvsdbConnectionManager.java

Change-Id: I272d716e3d8b3df9bdaea5fc0009c809c8e25933
Signed-off-by: rpujar <rpujar@inocybe.com>

Add yang for hwvtep

1. Addressed some review comments
2. Added some new fields
3. Moved file to hwvtepsouthbound-api
4. Fix for key mac-entry-key issue

Change-Id: I37945123170621dc8dacf187d2a47ff744fdceb5
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Merge "SecGroup: Handle cases when security group has empty rules"

Bug 3989: clear the connection info

When disconnecting, we need to clear the connection info to avoid
thinking we're still connected later.

Add assertions in the southbound IT tests to make sure we don't
connect twice.

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

SecGroup: Handle cases when security group has empty rules

Change-Id: I54ef43f6a4afd9fc571c8dcf9bf843007452c968
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Initial bundle creation for HWTEP SB"

Merge "A few cleanups"

Document how to use Docker Compose

Fig has been obsoleted by Docker Compose. Some distributions have now
replaced Fig entirely (e.g. Fedora 22); this patch documents Docker
Compose as an alternative to Fig.

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

A few cleanups

* use plain String concatenation instead of StringBuilder where appropriate
* simplify single-use return variables
* <> operator
* avoid calling new String()

Change-Id: I56e99426daeda620a37059417588e5ee4fb99c76
Also-By: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Revert "Use single-argument form of firstKeyOf()""

Revert "Use single-argument form of firstKeyOf()"

This reverts commit e19e4e3f10f65054aba16a94c082180f24b9beb6.

This is breaking the net-virt code. Bridges are not getting created when OVSDB server connects to the controller. Probably happening because of single argument firstKeyOf() method.

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

Initial bundle creation for HWTEP SB

1. Creating initial bundles for hwvtep southbound using Startup Project
archetype.
2. Adds hwvtepsouthbound bundle and entry in ovsdb/pom.xml

PatchSet2:
1. Addressed review comments and added hwvtepsouthbound-it

PatchSet3:
1. Fixed IT failure.

PatchSet4:
Fixed minor issue in southbound/default-config.xml

Change-Id: Ib185c1a9a90b08dc84f64231b8fee414d1e209a0
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Merge "Use single-argument form of firstKeyOf()"

Use single-argument form of firstKeyOf()

A few other clean-ups in the affected files:
* use plain String concatenation instead of StringBuilder where appropriate
* simplify single-use return variables
* <> operator
* avoid calling new String()

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

Drop obsolete dependencies

commons.httpclient, containermanager.it.implementation and
protocol_plugins.stub are no longer built by controller, and
don't appear to be used in ovsdb.

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

Added Security Rule for Custom ICMP

Change-Id: I5f813b3695c2fd21e03a0ccf215e49b891059154
Signed-off-by: sangeeta.maurya <sangeeta.maurya@hp.com>

Merge "Migration to use MD-SAL Project"

Merge "Sonar clean-ups"

Migration to use MD-SAL Project

Change-Id: Id34e216f57308f96139e95483b92df0e16e8c60a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "getDHCPServerPort is modified to address Bug 4206"

Sonar clean-ups

Starting from source files containing collapsible ifs:
* merge collapsible ifs;
* remove unused parameters;
* remove unused exceptions;
* remove unused fields;
* rename shadowing variables;
* add private constructors for utility classes;
* use more specific exceptions than RuntimeException;
* clean up "if () return true; else return false;";
* some Java 7 <> operators;
* add missing "static" for constants;
* remove unused private methods (except in OF13Provider);
* move some inline comments above the corresponding source line;
* used boolean instead of Boolean for Map::contains();
* replace foreach-style constructs examining the first element only
  with explicit "if (...hasNext()) ... = ... next()";
* prohibit overriding methods called from constructors;
* remove useless casts.

Reverse the order of node augmentation handling in SouthboundUtil so
that the augmentation case can actually trigger.

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

Use the library config bundle in southbound

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

getDHCPServerPort is modified to address Bug 4206

Change-Id: I121f91dadb370543664f4a235cf5d0eef6f8b932
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>

Merge "Migrate restconf dependencies"

Merge "Updating fix for bug 3989 based on discussion in OVSDB meeting"

Merge "Drop the #1997-related sleep"

Migrate restconf dependencies

Change-Id: I15dca264d53a6af9c80e46bc02c8cfab4709e0ee
Signed-off-by: Tomas Cere <tcere@cisco.com>

Updating fix for bug 3989 based on discussion in OVSDB meeting

Signed-off-by: Raksha Madhava Bangera <raksha.madhava.bangera@intel.com>
Updating the patch as per review comments

Signed-off-by: Raksha Madhava Bangera <raksha.madhava.bangera@intel.com>
Removing whitespace

Signed-off-by: Raksha Madhava Bangera <raksha.madhava.bangera@intel.com>

Added missing dependency on library-features.

features-ovsdb needs to have a dependency on library-features for
feature repository reference.

Change-Id: I52c741fb7a1dda6bf0f33b1717da9b05677ff1bb
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Clean up plugin management

A number of plugins are managed by odlparent, so use that instead of
variables (even inherited) and remove unnecessary entries (i.e.
specified identically in odlparent).

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

Merge "Switch from Felix to config-driven dependencies"

Switch from Felix to config-driven dependencies

The library module was regenerated using
opendaylight-startup-archetype 1.1.0-SNAPSHOT.

The ovsdb version is now 1.2.1-SNAPSHOT to make sure none of the
old 1.2.0-SNAPSHOT artefacts are used.

The library-dependent integration tests (at an OSGi level) are now in
library-it, apart from the plugin tests which are just ignored since
they're going away.

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

Merge "Use mdsal's base IT classes"

Bug 4045 - ipv6 work around

When running tempest, the following exceptions occurs because
ipv6 is not supported yet. This patch is work around for now.
TODO: support IPv6 and remove this work around

> 2015-08-25 15:00:12,428 | ERROR | ntDispatcherImpl | EventDispatcher                  | 400 - org.opendaylight.ovsdb.openstack.net-virt - 1.2.0.SNAPSHOT | Exception in dispatching event NorthboundEvent [handler=NEUTRON_ROUTER, action=DELETE, port=null, subnet=null, router=NeutronRouter [id=55dc4fd2-0e5f-4ec8-b589-18616dfaa6a1, name=tempest-router--44999886, adminStateUp=false, status=ACTIVE, tenantID=86d8df7573f24bc68e0a75846812839d, external_gateway_info=NeutronRouterNetworkReference [networkID=7d526ede-cfd0-45ec-897f-607471dd801e enableSNAT=true externalFixedIPs=[Neutron_IPs{ipAddress='172.24.4.62', subnetUUID='9a799aec-5f1f-4a2d-b3ed-a109f6c89bcb'}, Neutron_IPs{ipAddress='2001:db8::3e', subnetUUID='5178fb71-6683-4cf6-a1e7-095da5b9962a'}]], distributed=false, gw_port_id=2419ec70-21e8-4a3b-aa52-82ec9dd7f698, routes=[], interfaces={}], routerInterface=NeutronRouterInterface [subnetUUID=009ab797-30df-4c55-87f4-5142a9f8b444, portUUID=900abbc5-4db9-4bec-8777-0018f2950751, id=55dc4fd2-0e5f-4ec8-b589-18616dfaa6a1, tenantID=86d8df7573f24bc68e0a75846812839d], floatingIP=null, network=null, loadBalancer=null, loadBalancerPool=null, loadBalancerPoolMember=null]
> java.lang.IllegalArgumentException: Supplied value "fdfd:7da6:14e6:0:f816:3eff:fe43:c114/32" does not match required pattern "^(([0-9]|[1-9][0-9]|1[0-9][0-9]|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9][0-9]|2[0-4][0-9]|25[0-5])/(([0-9])|([1-2][0-9])|(3[0-2]))$"
>         at com.google.common.base.Preconditions.checkArgument(Preconditions.java:145)[39:com.google.guava:18.0.0]
>         at org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev100924.Ipv4Prefix.<init>(Ipv4Prefix.java:49)[49:org.opendaylight.yangtools.model.ietf-inet-types:2010.9.24.8-SNAPSHOT]
>         at org.opendaylight.ovsdb.utils.mdsal.openflow.MatchUtils.iPv4PrefixFromIPv4Address(MatchUtils.java:1133)[401:org.opendaylight.ovsdb.openstack.net-virt-providers:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.services.L3ForwardingService.programForwardingTableEntry(L3ForwardingService.java:64)[401:org.opendaylight.ovsdb.openstack.net-virt-providers:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programL3ForwardingStage2(NeutronL3Adapter.java:689)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programL3ForwardingStage1(NeutronL3Adapter.java:670)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.updateL3ForNeutronPort(NeutronL3Adapter.java:640)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleNeutronRouterInterfaceEvent(NeutronL3Adapter.java:310)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.RouterHandler.processEvent(RouterHandler.java:208)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.dispatchEvent(EventDispatcherImpl.java:95)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.access$200(EventDispatcherImpl.java:28)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl$1.run(EventDispatcherImpl.java:57)[400:org.opendaylight.ovsdb.openstack.net-virt:1.2.0.SNAPSHOT]
>         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)[:1.7.0_79]
>         at java.util.concurrent.FutureTask.run(FutureTask.java:262)[:1.7.0_79]
>         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)[:1.7.0_79]
>         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)[:1.7.0_79]
>         at java.lang.Thread.run(Thread.java:745)[:1.7.0_79]

Change-Id: If886c3867665660c8f90a72f567420f48f3fd6ad
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Merge "Added unit tests for ovsdb/southbound"

Merge "Fix for defect 4135"

Added unit tests for ovsdb/southbound

Change-Id: Iab67bab14a90ef39678756e7bd9c8852bf111f40
Signed-off-by: rpujar <rpujar@inocybe.com>

Merge "Added Unit Tests for ovsdb/southbound"

Bug 4160 - null pointer exception in getDHCPServerPort() (cont)

Avoid null pointer exception in when neutronPortCache.getPort() returns null.

Patch set 2..4: Add logs to make code same as what is in stable/lithium

Co-Authored-By: Isaku Yamahata <isaku.yamahata@intel.com>
Change-Id: I41630cd978e68cbd135634646f3e3155f14715c7
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Bug 4160 - null pointer exception in getDHCPServerPort() (re-do)

null pointer exception in SecurityServicesImpl.getDHCPServerPort()
because neutronPortCache.getPort() may return null. It needs null check.
NOTE: the order of neutron port creation/deletion via neutron northbound
api and ovs port creation/deletion is not deterministic. Either order can
happen. SecurityServicesImpl seems to be fixed to be aware of it.
it's another story. This patch addresses only null pointer exception
for now.

This is a redo, to fix a merge mistake introduced in
https://git.opendaylight.org/gerrit/#/c/24579/8..9

Original-Change-Id: I40f78b1477f1c2a6ad3e23b2e60e21be6d9abc23
Change-Id: Ib929bf774426d6240934ea406189e25be1ecf536
Co-Authored-By: Isaku Yamahata <isaku.yamahata@intel.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Replace StringBuilder with more efficient String concatenation (re-do)

Just replaces the StringBuilder with a more efficient/readable
String concatenation operation.

This is a redo, to fix a merge mistake introduced in
https://git.opendaylight.org/gerrit/#/c/24579/8..9

Original-Change-Id: Idce8fed3d4e207ff1a95cf24267a41fa8682a08a
Change-Id: I9aa8d648398e532ad9eacd9a07b99b672ebfebd6
Signed-off-by: Flavio Fernandes <ffernand@redhat.com
Co-Authored-By: Ryan Goulding <ryandgoulding@gmail.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

fix up of 474eb2a9d61bb1f2f4664d12c23042ab3699260b

The changeset of 474eb2a9d61bb1f2f4664d12c23042ab3699260b
Change-Id: I442745135b5ccb778b1f0b11962f50b1f2d5ff5b
silently reverted the patch of d3d11112b0496535a85ba9dee3831c341c5e4054
Change-Id: I900e3ab959917fda35dd5f073b0cf89b32edc031
This patch introduce the fix again.

Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>

Fix for defect 4135

https://bugs.opendaylight.org/show_bug.cgi?id=4135

If integration bridge or external bridge exist prior to
connecting to ODL, then the pre-existing bridge is not added.
But the side effect is that the controller is not set on
this bridge.
Current code checks to see if the bridge already exists on the node. This
patch adds a check to see if the bridge exists on CONFIG datastore. If the
bridge does not exist on config, it indicates that the bridge existed prior
to ODL connection, and it adds the bridge.

Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>

Added Unit Tests for ovsdb/southbound

InstanceIdentifierCodecTest.java and OvsdbSchemaContantsTest.java

Change-Id: I91e356dfc6bfc41bac081b9792ff99bfc931c3ae
Signed-off-by: rpujar <rpujar@inocybe.com>

Use mdsal's base IT classes

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

Added IPV4, Custom TCP and UDP SG rules.

Patch set 9: undo mess in patch set 8, making it the same as patch set 7

Change-Id: I442745135b5ccb778b1f0b11962f50b1f2d5ff5b
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>

Merge "Bug 4163 - null pointer exception NeutronL3Adapter.getExternalNetworkSubnet"

Merge "Replace StringBuilder with more efficient String concatenation"