Merge "Sonar clean-up: OF13Provider"

Merge "Add libvirt support"

Add libvirt support

Set up memory sizes with libvirt. I'm not familiar enough with Ruby to
know if there's a way of combining the :virtualbox and :libvirt
stanzas...

(The box images still need to be mutated manually.)

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

Update showOvsdbMdsal.py to display bridge name and mac next to alias

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

Sonar clean-up: OF13Provider

This class contains lots of methods with unused parameters, but they
follow a pattern so override that instead of cleaning up the
parameters.

Dead-code elimination (unused methods, unnecessary initialisations).
Code which corresponds to comment method calls is marked as such so
it no longers causes Sonar warnings (with a TODO added to check the
call sites).

getTunnelReadinessStatus() appears unused but is tested for in the
corresponding unit-test class, so keep it for now and override the
unused warning.

Long dpid -> long dpid.

if (!local) {} if (local) {} -> if (!local) {} else {} (the value of
local doesn't change in the first block).

networkingProviderManager is initialised and used in a single method,
so convert it to a local variable and adjust the corresponding test.

Return value simplification (? true : false isn't necessary).

<> operator in OF13ProviderTest.

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

Bug 3378 - ovsdb netvirt needs help in getting mac for a given ip in br-ex

This patch, does not do periodic refresh of gateway mac. Will push that in next gerrit.

Change-Id: Ie1a5a81fb7f196874f87bfbe0095ea595756a0a4
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 65a8de37c522f8f09caf6be73997906b98fd0b89)

Fix it failure: Error resolving artifact

Fix the following issue:
error: java.io.IOException: Error resolving artifact org.opendaylight.ovsdb:southbound-karaf:zip:1.1.0-SNAPSHOT

In it builds, such as:
https://jenkins.opendaylight.org/releng/view/ovsdb/job/ovsdb-daily-full-lithium-integration/OVS_VERSION=2.3.0,jdk=openjdk7,nodes=dynamic_docker/

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

Merge "Sonar clean-up: braces for control statements"

Merge "Added fixed DHCP security rules, which will be added on a VM create."

Sonar clean-up: braces for control statements

Add braces for if, while, for statements. (Automated, thanks IntelliJ!)

In a few cases a different change was more appropriate:
* ConfigurationServiceImpl.java: in unsetOvsdbConfigurationService(),
  the if is redundant, we might as well set to null unconditionnally.
* ConnectionServiceImpl.java: likewise
* OvsdbLibraryIt.java: instead of
  if (...) { return true; } return false;
  just
  return ...;
* OvsdbVswitchdSchemaConstants.java: likewise

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

Added fixed DHCP security rules, which will be added on a VM create.

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

Clean-up in BridgeOperationalState

The main fix is moving null-checks on iid before their first use.

db and changes are only used in the constructor, they don't need to
be in the instance; since the class isn't serializable removing the
fields doesn't change anything external.

Java 7 improvements: <> operator and merged exceptions.

Use Optional.fromNullable() to simplify some Optional creation code
and avoid double calls to getAugmentation().

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

Merge "Fix Javadoc error."

Fix Javadoc error.

[ERROR] /Users/adetalhouet/Documents/odl/Beryllium/ovsdb/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/NeutronL3Adapter.java:288: error: @param name not found
[ERROR] * @param action the {@link org.opendaylight.ovsdb.openstack.netvirt.api.Action} action to be handled.

Java version "1.8.0_45"

Change-Id: I460cb898c3a02678ecd4f58281bd289d607673e3
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add showOvsdbMdsal.py tool

This tool can provide useful info in regards to ovsdb's mdsal structures.
It looks at the operational or config trees in mdsal, depending on the
parameter '--config':

Patch set 2: code review

$ ./showOvsdbMdsal.py -h
Usage: showOvsdbMdsal.py [options]

Options:
  --version             show program's version number and exit
  -h, --help            show this help message and exit
  -d, --debug           Verbosity. Can be provided multiple times for more
                        debug.
  -n, --noalias         Do not map nodeId of bridges to an alias
  -i ODLIP, --ip=ODLIP  opendaylights ip address
  -t ODLPORT, --port=ODLPORT
                        opendaylights listening tcp port on restconf
                        northbound
  -u ODLUSERNAME, --user=ODLUSERNAME
                        opendaylight restconf username
  -p ODLPASSWORD, --password=ODLPASSWORD
                        opendaylight restconf password
  -c, --config          parse mdsal restconf config tree instead of
                        operational tree
  -f, --hide-flows      hide flows

Examples of what to expect:

  https://gist.githubusercontent.com/anonymous/3a952cec49ef92461752/raw/7abfedcd7790acc5b5cef07a2c08da0405f303c6/gistify465337.txt

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

Merge "Update README to reflect LBaaS postman collection."

Merge "Change log level of raw json rpc message"

Change log level of raw json rpc message

When hypervisor/hardware vtep are not implemented in the right way and sending jsonrpc message with wrong format, the log of raw json rpc message helps debugging.

Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>
Change-Id: I033abdb251065dd19c8ee198417f04bcd152e761

Adding more documentation/comments to the code.

Change-Id: Ieb9b84fd53b400b3c01c5b583337dd480b2207c1

Signed-off-by: Marcelo C. Amaral <marcelo.amaral@bsc.es>

Update README to reflect LBaaS postman collection.

Change-Id: I53057cd71fed174debbf1e6be579fb36e3d76b41
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add postman collection to interact with LBass.

This postman contains queries to interact with:
- pool
- pool member
- loadbalancer

It also contains queries to create a network, a subnet and two ports, in order to have a proper network topology for testing purposes.

Change-Id: I9bc1fd6e7d59abea5b22bb8260d79b9fa11e9dc9
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Bug 3962: Event dispatcher found no handler for NorthboundEvent (part 1)

We cannot reproduce this locally, so this commit adds some better logs
to inform what handlers have this issue.

Patch set 2: code review.
Patch set 3: remove redundant param on the non-error path: ref.

Change-Id: I394984cf41f7e310919efdb7c4bbf394ba1fa89d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit 0b6e3a67980f9e30ba9cf07ddd0fd2d1c6664e1f)

Merge Sonar fixes from stable/lithium..

Change-Id: I03d18ba2dcaa76de565d53bc9dd68f9b28c2b240
Signed-off-by: Andre Fredette <afredette@redhat.com>

Merge "bug 3909: Disable SSLv3 connection for ovsdb server"

Bug 3954: NPE exception in isDistributedArpDisabled()

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

bug 3909: Disable SSLv3 connection for ovsdb server

SSLv3 has known vulnerability.
For security reason, we should disable sslv3 protocol for ovsdb server.

Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>
Change-Id: I1267711efe7cebcd3252d81f184aab8782b6d72d

Merge "Add ODL controller ready check before running robot tests"

Add ODL controller ready check before running robot tests

Change-Id: I8ee340873ee47af89373afe5ed81bcff787a524a
Signed-off-by: Ben Eze <beze@inocybe.ca>

Merge "Update NeutronL3Adapter UT"

Merge "Sonar bug fix: block of commented-out lines of code should be removed"

Sonar bug fix: block of commented-out lines of code should be removed

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

Update NeutronL3Adapter UT

Patch Set 1: initial commit
Patch Set 2: some more to go
Patch Set 3: some more tests - done with the class

Change-Id: I1081c7e70e7e437ac6260624c7f1b0cc4a5ceeae
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Update OF13Provider UT

Change-Id: Idba5421b9a4f6e63e713f368c8dd3011c5c11819
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Bug 3893: provide well known url to know that net-virt topology is ready

With this change, one can check the following url and know that ovsdb's netvirt
is up and functional:

   http://${ODL_IP}:8080/restconf/operational/network-topology:network-topology/topology/netvirt:1

Patch set 2: fix transaction
Patch set 3: code review, move new function to OvsdbInventoryServiceImpl
Patch set 4: typo (functional)

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

Merge "Added the robot framework to OVSDB VMs"

Added the robot framework to OVSDB VMs

Change-Id: Ib57793b05661d5d5487788574ef65cfa990d3cb4
Signed-off-by: Ben Eze <beze@inocybe.ca>

Fix Javadoc errors.

The build was failing due to those errors:
[ERROR] /ovsdb/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/api/EgressAclProvider.java:13: error: @param name not found
 --> doesn't exist
[ERROR] /ovsdb/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/api/IngressAclProvider.java:22: error: @param name not found
--> doesn't exist
[ERROR] /ovsdb/openstack/net-virt-providers/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/AbstractServiceInstance.java:205: error: @param name not found
--> Node instead of node

Java version "1.8.0_45"

Change-Id: I386bb5ba6be41c134c6f843f78ef06147528fd61
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Sonar bug fix"

Bug 3841 - wrong severity level: getTenantNetwork did not find network

when using normal ovsdb net-virt, I see benign logs like these:

WARN  | ntDispatcherImpl | TenantNetworkManagerImpl | 271 - org.opendaylight.ovsdb.openstack.net-virt - 1.1.0.SNAPSHOT | getTenantNetwork did not find network for patch-int

these should be debug at best; not warning.

Patch set 2: add changes from code review

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

Bug-2094 : less is more remove redundant code

In pipeline, there is no need to set register value to 0,
as that is the expected value upon classifier table.

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

Bug 3831 - ignore unknown OVS interface and datapath types

Ighore OVSDB interface or datapath types which are not
present in the MD-SAL southbound model (but are present
in the OVS nodes open_vswitch table iface-types or
datapath-types columns).

Tested by making a build of ODL which removed supported
types from the ODL code to varify the log warnings
were created.

Change-Id: I05e689fa2b2dc906e3083e4fc44ba60abdb1d352
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
(cherry picked from commit c525fd2b379171af656ffd060d9e520ab4e3854e)

Bug 3831 - add stt interface type to model

On a build from openvswitch master branch, there was an
interface type 'stt' which was not included in the ovsdb.yang
model for the southbound MD-SAL.  When an OVSDB connection
was made to this OVS node, the operational md-sal failed
to populate.

Adding the 'stt' interface type to the model is a quick
fix.  A better fix would ignore types not present in the model.

Change-Id: Idf259998f73d43be587d19de73794a174d4d81d3
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
(cherry picked from commit 444b68f6f7637d9b4eb7e4f07c9ff6ced4e8ce76)

Sonar bug fix

Removing redundant 'public' keyword in interface in ConfigurationService.java

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

Remove frame size limit for scale OVS

Change-Id: I4f1c7e29c332964eea077fd6958453be34fa04a5
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Bug 3647 :Operational not set correctly when other-config is removed from switch.

similar to BZ 3647, but for other-config

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

Bug 3647 : Fix for bugzilla 3647

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

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

Bug-2094 : L3 North-South does not work -- fix outbound rewrite

This set of changes will fix the outbound table (ie table 100) to
match on fixedIp associated with floatingIp, destined to neutron
router's mac. Packets that match the criteria will have rewrite
done, and be sent out of the patch-port in br-int.

Pat set 2: Fix copy and paste mistake in testProgramIpRewriteRule()

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

Merge "Added Vagrant images to run CSIT."

Added Vagrant images to run CSIT.

Change-Id: I147cb90878f8f8af6d535b2b15988e9221777a4c
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
Signed-off-by: Ben Eze <beze@inocybe.ca>

Bug-2094 : L3 North-South does not work -- fix inbound rewrite

This set of changes will fix the inbound table (ie table 30) to use reg3
to track the ip rewrite. Also, changes added to routing table (talbe 60)
so that reg3 match can route packet into tenant segmentation.

Patch set 2: code review.
Patch set 3: fix test to account for removal of inboundIpRewriteExclusionCache.

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

Bug-2094 : L3 North-South does not work -- fix ARP table

This set of changes will fix the ARP responder table (ie table 20) to properly
handle the responding of ARP queries made on br-ex for the associated floating
ip addresses.

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

Merge "Skeleton for the ovsdb-ui feature"

Skeleton for the ovsdb-ui feature

Folders structure and basic files to create and add a external
module on DLUX. It also include a CRUD implementation of
the southbounds REST API.

Change-Id: If48eb4d055db3b91529e76e75503aa4c68b0c6a3
Signed-off-by: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>

Bug 3752 : NPE at SouthboundMapper.java -- createControllerEntries

Check that ovsdbBridgeAugmentation.getControllerEntry() is not
returning null

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

Bug 3379:java.lang.IllegalArgumentException: ControllerEntryKey

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

Bug 3379:java.lang.IllegalArgumentException: ControllerEntryKey

Problem: OVSDB Controller updates are indpendent of Bridge updates but the ovsdb.yang model includes the controller data in the OVsdbBridgeAugmentation. This existing code assumed that the Controller and Bridge updates would be together and thus would always have the OvsdbBridgeAugmentation to update. That is not the case since the Controller updates can be received without a Bridge update.

Solution: Add the ControllerEntry to the OvsdbBridgeAugmentation using the Bridge if it was included with the Controller update. Otherwise, find all the OvsdbBridgeAugmentations for the connection and update them if they are using the updated Controller.

Patch also includes updates to SouthboundIT and NetvirtIT. NetvirtIT was needed so that the isConnected=true case would be tested. Southbound does not include the openflowplugin but Netvirt does. The openflowplugin is needed to test when the switch connects to the openflow controller.

Other fixes were made to the existing testOvsdbBridgeControllerInfo(). That test was using the ovsdb node ip:port as the controller target but it should have been the controller's ip:port.

Change-Id: I30785dbdf955614ad6fc675507da048434c2485e
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit fd1b45b2100fb2bfe7a7b58295cebdc949833680)

Merge "Bug 3681 - Uncaught ConnectException leaves library in bad state"

Merge "Bug 3681 - Uncaught ConnectException leaves library in bad state"

Merge "Separation of ARP Responder from config.ini. Amended to use ConfigurationService & fix comments"

Separation of ARP Responder from config.ini. Amended to use ConfigurationService & fix comments

Change-Id: I2dac1bec77bdc9818489dc7773a8804464a7bad5
Signed-off-by: evvy <dhiraviam.natarajan@gmail.com>

Fix Bug 3663: Wrong assert statement causing test failure.

Also remove a System.out than shouldn't have been there in a first place.

Change-Id: Iad4a3968451d57ff7fe4d37ea380f583ffc9d3c7
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Adding test case to set-target controller for a bridge and verify if it is set in the bridge as expected

Change-Id: I85b29409178716920c6e0859580f035055a0901c
Signed-off-by: Praveen Mala <praveen.mala@intel.com>

Merge "Fix Bug 3663: Update netvirt.provider UT"

Bug 3681 - Uncaught ConnectException leaves library in bad state

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

Bug 3681 - Uncaught ConnectException leaves library in bad state

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

Merge "Bug 3141 - OVSDB southbound operational MDSAL stops getting populated after ovs-vsctl del-manager"

Merge "Bug 3676 - OvsdbNodeRemoveCommand .. had no OvsdbNodeAugmentation"

Bug 3141 - OVSDB southbound operational MDSAL stops getting populated after ovs-vsctl del-manager

The symptom looked like the OvsdbDataChangeListener#onDataChanged() was not being called when the node was added back to config (after it was deleted from operational followed by deleted from config)

This was because of a AbstractFuture.get() was not returning. This was a get() for the transaction responses for ovsdb transactions. In this particular path the node was deleted from oeprational. Then the node is deleted from config. During the processing of the config update there was an expectation that some ovsdb updates would happen but there aren't any because the node was already removed from operational and there is no active connection to the ovsdb node. This caused the invoker thread to block on the future result.get(). Later when the node was added to config, the onDataChanged() was not called since the thread was still blocked on the get().

Solution: check if there were any ovsdb transactions and not call get() if there aren't any.

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

Bug 3676 - OvsdbNodeRemoveCommand .. had no OvsdbNodeAugmentation

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

Fix Bug 3663: Update netvirt.provider UT

This patch update all the test classes within the
org.opendaylight.ovsdb.openstack.netvirt.provider package

Change-Id: I40b73ae5efe472449b2b3a11e91dbef6d5bf24cc
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Fix Bug 3663: Update netvirt.impl UT"

Merge "Fix Bug 3663: Update netvirt UT"

Fix Bug 3663: Update netvirt.impl UT

This patch update all the test classes within the
org.opendaylight.ovsdb.openstack.netvirt.impl package

Change-Id: If9864fc98385c602627b279653875875fdcf700c
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Fix Bug 3665: Call the nodeCache.remove() by passing the NodeId.

Change-Id: I7b55ef7e86c76f473e9dc044f5441d989034a52b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Fix Bug 3663: Update netvirt UT

This patch update all the test classes within the
org.opendaylight.ovsdb.openstack.netvirt package

Change-Id: I8f747293f9283bc4f3c0823f4504a63dd59a03c0
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Change EventDispatcher field visibility of AbstractHandler to protected.

Instead of having EventDisptacher private, we set it as protected thus
all the classes that extends AbstractHandler can use the variable.
This patch remove:
- the setDispatcher() method from AbstractHandler
- the declared EventDispactcher field of the classes extending AbstractHandler

Change-Id: Idbd000def09aa91cc4e40976ac4a37274bfce762
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Bug-2094 : L3 North-South does not work

Create br-ex and add patch port between br-int and br-ex when L3 fwd is enabled.

Patch set 2: Use consts for patch port names
Patch set 3: Use config to determine if l3 fwd is enabled
Patch set 4: Fix tests part 1 of 2
Patch set 5: Fix tests part 2 of 2
Patch set 6: Add changes from code review

Change-Id: I2425d83c68025617a50000f98c4e60b3a9a15ee8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Also-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 6940c8d516345a3e9b11e148830d881053edfde7)

Bug 3310 Bug 3316: Fix for iids for termination points

Change-Id: I9174fe480fc95570e52095474c3e9ae78b8efd8e
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 5c5ed0bec9b9f908d97d024549cb754f3898d5eb)

Bug 3310 Bug 3316: Fix for iids for bridges

Change-Id: Ieabaa7040cbcdc8e72a2aae21a9181667c543880
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 004394b0b3da86ddb506c757b8c5f110d916b02a)

Bug 3310 Bug 3316 Use iid stamp if we configured a connection, otherwise systemid

This has been tested, and passes  SB IT tests (all 21)
As well as manually tested beyond their coverage.

PatchSet3: Switch to using ovsdb://uuid/<uuid> for default
NodeKey for OPER nodes we discover that we did not configure.

Conflicts:
southbound/southbound-it/src/test/java/org/opendaylight/ovsdb/southbound/it/SouthboundIT.java

Change-Id: I270a64e13a6fca257b49d2e2ff34f754698c2df4
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Slight refactoring of the lifecycle of data"

Merge "Bug 3563 - Fix/clean methods that determine the OpenFlow Controller ip address"

Merge "Add per test information"

Slight refactoring of the lifecycle of data

With this patch, we change slightly the manner in which we handle
data change events (including connections)

Previously, when we got a DataChangeEvent we'd try the following

1)  connect(changes) - do all the work for connecting, including
     registeringCallbacks
2)  connectionUpdate - handle connection updates
3)  updateData - use transact to write any data changes to the ovsdb instance
4)  disconnect - disconnect anything if needed

Now we have changed this so that
a)  #1 (connect) - does *not* registerCallbacks
b)  Added a 5) registerCallbacks

The reason this was done is so that the normal pipeline for writing data
can handle flushing the externalId for the OpenVSwitch table down to ovsdb
*before* we registerCallbacks and have it come back to the operational data
store.

Conflicts:
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/OvsdbConnectionInstance.java

Change-Id: I105dd01bf87b1be170e2c4082a7914fba2002249
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Write iid of ovsdb-node to OpenVSwitch.externalIds"

Write iid of ovsdb-node to OpenVSwitch.externalIds

Change-Id: I4fbdd5499b19d452f253a3600944aecced59dafc
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 3747b449b15deccb33d152924c492647a92c8d25)

Switch from using OvsdbCreateCommand to using OpenVSwitchUpdateCommand

Have the initial registration of the ovsdb-node come from getting
the OpenVswitch table data come to use from the monitor command in the
initial db dump.

This passes all SB-IT tests:

Tests run: 21, Failures: 0, Errors: 0, Skipped: 0

Change-Id: If098081bb89292f0bac320659b41fa05994e195e
Signed-off-by: Ed Warnicke <hagbard@gmail.com>

Pass OvsdbConnectionInstance to Commands instead of ConnectionInfo

This is in preperation to have the adding of ovsdb-nodes handled
by the OpenVSwitchUpdateCommand, so it can write the iid to the
externalIds.

Change-Id: Ib478c776e0906e7424b5759422d0be986dccdc29
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
(cherry picked from commit 02c8562dfca6d7863ae6cdf34ef502d6a4ad370d)

Bug 3563 - Fix/clean methods that determine the OpenFlow Controller ip address

Change-Id: I420a10e769be016efa0a4e6c71d863fd39c12e23
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 94dbb902c1f020c87407ba0fb87ff770dea01cbc)

Add per test information

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

Fix for bug 3538

in removeOutputPortFromInstructions function,
when reordering the actionList setKey should be used same key with order.

Change-Id: I3ab3b6c9bbf6bf0b3914dc0819501177f37dc31a
Signed-off-by: shc411 <jung33@gmail.com>

Remove northbound and compat it tests

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

Bug 3332: Fix connections made to an ODL instance

If an OVSDB connection is made via OVSDB SB MD-SAL
to self (same ODL) or another ODL instacne, the
OVSDB SB provider would hang or otherwise stop working.
This fixes by returning a null list to the 'list_dbs'
command.

v2 - remove white space
     When an OVSDB connection is established,
     the first request that the OVSDB Southbound makes is a
     list_dbs command.  By returning an empty db list, two
     things occur:
     1 - the request gets a response
     2 - the empty db list signals an uninteresting connection

Change-Id: I295dd47b821f7ec4ead0eb489a1b124cc574e8cf
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

Merge "Fixing sonar issues on ConfigurationServiceImpl.java"

Bug-3425 : Cont. minor changes to incorporate the code review from Sam

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

Bug-3425 : VM doesn't get the ip address from DHCP
server, if network is not attached to the router

This failure happens when net-virt don't find tunnel port in
the Node update from operational data store.
It tries to extract termination point from node update which
doesn't have tunnel port termination point. And because
there are other termination point exist, it doesn't go and
read from data store. ProgrammeTunnelPort() method iterate
through these existing termination points and it doesn't find the
tunnel port and it returns without installing the Local Ingress
Bridge Rules. This patch first try to create a tunnel configuration
on integration bridge of both end node of the tunnel and then
installs the tunnel flows. Also while installing tunnel flows
It specifically look for tunnel port in the nodes update and  if
it doesn't find that it reads it from data store.

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

Fixing sonar issues on ConfigurationServiceImpl.java

Change-Id: Ic042ebc8605b23eea7d8a9db089f0cf6b1206e9b
Signed-off-by: Leo Cavalcanti <leozinho@gmail.com>

Remove adsal-related modules from features

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

Incrementing versions by 0.1.0 for post-lithium master branch

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

Merge "Add sonar report to SouthboundIT"