GET API uses @Consumes annotation incorrectly, while it should be @Produces.

Change-Id: I6c05abf0af436db0bdaceb9a94db17345e980353
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Fixing a couple of Integration distribution issues due to the legacy vs MD-SAL openflowplugin conflicts.
After the introduction of NX-Extension support, the OVSDB project has moved from a soft-dependancy (via MD-SAL Apis)
to a more hard-depedency with Openflowplugin/Openflowjava projects. Due to that reason, the filtering done during the
[-of10|-of13] switch for legacy vs MD-SAL plugin selection needs some improvement and that is taken care in the
integration project using https://git.opendaylight.org/gerrit/#/c/10003/.

Unfortunately, the net-virt-providers currently carries both OF10, OF13 providers together and hence, filtering that
bundle is not an option based on the above flag switch. Hence, added optional-dependency directive to make sure the
OSGi runtime dont scream when it finds a missing bundle for NX-Extension when running in legacy -of10 mode.

Better fix would have been to reorganize the providers to their own bundles and fitering them out using the above switch.
But since Integration project is impacted, this gives a good smooth transition to the newer model.

Change-Id: I11992a2ba872a0ac49c8a815dc7b4a5bce354b41
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added Karaf Shell framework for OVSDB plugin and printCache command-line.

More commands to follow soon.

Change-Id: Iee05b833d8d59394bf4ce09d99a9d85322e110c9
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Removed some unused files and dependencies.

Change-Id: I58312ab8f683fc7c81bdbf89811bee0a806ac0d9
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added the missing TearDown for HardwareVtep IT.

Change-Id: I170fd69f1a1cb1b44ae0bc4c36fef7072aec6783
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Moved MD-SAL openflow utility classes from net-virt-providers to its own Utility library that can be embedded into various applications.

Change-Id: I87ff6ae4e61e1859bf0a32d76efc98fb483899cf
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Added Nsi Utility method & resolved a multi-augmentation extension issue"

Merge "NXM_NX_NSI openflow extended match support."

Merge "Augment postman neutron collection and added FloatingIPHandler skeleton."

Merge "Utility method in net-virt-providers to set a NSP match."

Merge "NXM_NX_NSP openflow extended match support."

Added Nsi Utility method & resolved a multi-augmentation extension issue

If a MatchBuilder carries more than 1 extension, the last extension overrides all others.
Resolved the issue by creating an extensionList as and when the MatchBuilder is populated.

Change-Id: I0ef4260fd1635bd57f09b2c5e72aee970ce4e829
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

NXM_NX_NSI openflow extended match support.

Sample output:
[root@localhost ~]# ovs-ofctl dump-flows br-int -O OpenFlow13
OFPST_FLOW reply (OF1.3) (xid=0x2):
 cookie=0x0, duration=3.639s, table=1, n_packets=0, n_bytes=0, send_flow_rem ip,nsp=1000,nsi=250,nw_dst=1.1.1.1 actions=set_nsp:0x3e8,set_nsi:45

Change-Id: I449b8179add3eb164eab45e97f8886cb4d8ed23f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Augment postman neutron collection and added FloatingIPHandler skeleton.

Added more crud operations to postman neutron collection: floating ip.
Added FloatingIPHandler (which will be needed for L3 forwarding feature).

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

Utility method in net-virt-providers to set a NSP match.

Change-Id: I9f2c3611b0b16c78d91615968509a0f93adf50a0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

NXM_NX_NSP openflow extended match support.

Sample output :
[root@localhost ~]# ovs-ofctl dump-flows br-int -O OpenFlow13
OFPST_FLOW reply (OF1.3) (xid=0x2):
 cookie=0x0, duration=12.100s, table=1, n_packets=0, n_bytes=0, send_flow_rem ip,nsp=1000,nw_dst=1.1.1.1 actions=set_nsp:0x3e8,set_nsi:45

Change-Id: I5c88114448972f9b5f561d438119b17b2e013a28
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Augment postman neutron collection and added RouterHandler skeleton.

Added more crud operations to postman neutron collection: router.
Added RouterHandler (which will be needed for L3 forwarding feature).

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

Merge "Removed the unused netconf config files (which are auto-populated using maven-dependency plugin) from the distribution resources folder"

Merge changes Ic6cb8725,Ie533ed3e

* changes:
  Added utility methods for set_nsp and set_nsi actions.
  set_nsp and set_nsi action NX support

Removed the unused netconf config files (which are auto-populated using maven-dependency plugin) from the distribution resources folder

Change-Id: I3e44c6d8141753f4b048d4f1f76c5a5964510644
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added utility methods for set_nsp and set_nsi actions.

Change-Id: Ic6cb8725f394251eb6f89e95e5ee8d9428468e62
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

set_nsp and set_nsi action NX support

Sample output from OVS:
[madhu@localhost ~]# sudo ovs-ofctl dump-flows br-int -O OpenFlow13
OFPST_FLOW reply (OF1.3) (xid=0x2):
 cookie=0x0, duration=20.337s, table=1, n_packets=0, n_bytes=0, send_flow_rem ip,nw_dst=1.1.1.1 actions=set_nsp:0x3e8,set_nsi:45

Change-Id: Ie533ed3eaca7f032a36bf19230a22004d75db424
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added more crud operations to postman neutron collection

added: port, subnet, network.

Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Added some useful flow-mod utilities including extensions from other projects into of13provider bundle.

Change-Id: I465deba06ebf5068aec69bfc643f45a3c97f570a
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Introducing Openflow NX Extension infrastructure and NX_RESUBMIT action in ovsdb project

- This infrastruture will make use of the openflowplugin's extension infrastructure to add to the Extensibility tree.
- Few of the NX actions and matches will be introduced here and ideally all the NX actions should reside in a OVS friendly place.
- NX_RESUBMIT and NX_RESUBMIT_TABLE codec are added and the appropriate action type will be used based on the data passed by the application
- Fixed Distribution to make use of this infra.

Change-Id: Ibdc634afe7704c9a4890a952ddb4bff5ef066610
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Pull in the NXM extension infrastructure support into ovsdb distribution

Change-Id: Id623088f6bd20ab121a53009156a90fe92a655d0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Adding JavaDoc to one of the IT cases to demonstrate the Sonar IT Documentation

Change-Id: I61380c1f95edef3029a4eb2fe29e00c1381204d3
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

refactoring ovsdb of13 neutron

patch#5:
renamed openflow constant

patch#4:
rebase

Patch#3:
-rebased, added a couple of constants and deleted old
commented out code from Hydrogen.

-I am getting so many node errors on the current
master that it is hard to tell if I am introducing
any new ones. We should probably verify the existing base
isnt broken for all of us before we merge this until md_sal
starts behaving like it did prior to all of the
normalization/node work by that team captured in Bug #1490.

Patch#2:
Changed a couple assessors in the match and comments
in instructions

-First of a few, was wanting  to put a builder wrapper around
all of the ofplug java API boiler plate sounded good in
theory but maintenance and tshooting mdsal RPCs,
threading etc from a loosely coupled wrapper may eventually
make us sad pandas.

-Pulled the ofplugin match/action APIs out of OF13Provider
and into a seperate package. Will work with Dave/Madhu
(or anyone else up to speed on karaf strategy) to package
it into Karaf or something if we are still aiming for
seperate bundles.
-Want to break it up into a progressive set of patches rather
then one biggy since we dont have IT yet to avoid any
impact.
-I have tests done just no PaxExam to run them against.
Can push them as a stub/example if someone wants.
-Added TCP_Flag and IPv4 Tunnel_Src/Dst.
-Modified and added a couple of constants.
-Later today will be adding a few methods to remove some
boiler plate jsut not as much as initially desired but
instantiating the MDSAL builders outside of of13provider
was getting mixed results with lots of rpc errors. Will
push it with a bug tomorrow anyways as a talking point for
out consumer APIs moving forward.

Change-Id: I05d7cccf3d7d0ae481070d8410e2328588faa348
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Add Openflowplugin Extension API to OVSDB Distribution

Change-Id: I395549123fba6d85d0f9e35ac3b853c9cc081522
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge "Wait for result of Transactions in MD-SAL Adapter"

Merge "Close the SouthboundHandler Executor Gracefully"

Wait for result of Transactions in MD-SAL Adapter

This patch waits for the result of a transaction to return before
executions continues. Additionally, logs have been cleaned up

Change-Id: I3602bd3784af1b3c290efc58a7d6018687441fc3
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge "Bug 1396 : Change from deprecated DataBrokerService to DataBroker"

Close the SouthboundHandler Executor Gracefully

During the OSGi "stop" method try and close the Executor gracefully.
Stop accepting new work by calling shutdown()
Wait a given time for  the current task to terminate before issuing a
shutdownNow()

Fixes bug 281

Change-Id: I0823a3dce87e67ca3e5d18640e9ecd5d4e75b7ba
Signed-off-by: Dave Tucker <djt@redhat.com>

Remove reference to stale model-flow-management

- Remove reference to an artifact that is no longer created in
controller
- Fixing a whitespace issue on the feature.xml file that could cause
issues downstream on a build

Change-Id: I6fa09234a23760220e32df689d0728b579ba34f6
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Bug 1396 : Change from deprecated DataBrokerService to DataBroker

Script for creating coke tenants in devstack:
https://gist.github.com/1ba897e7cde8dbd87fe8

Starting odl command:
./run.sh -Xmx2048m -XX:MaxPermSize=512m -virt ovsdb

LogLevel:
setLogLevel org.opendaylight.ovsdb.openstack.netvirt.providers.OF13Provider TRACE

Log of test:
TBD

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

Merge "Add MD-SAL Adapter for the OVSDB Plugin"

Add MD-SAL Adapter for the OVSDB Plugin

This commit adds an initial MD-SAL connector for OVSDB
The OvsdbInventoryManager updates the MD-SAL with nodes as they are
added and removed from inventory. The Yang model provides a reference
between OVSDB nodes and the OpenFlow nodes (bridges) that they manage

Also changes the nodeAdded API in OvsdbIventoryListener to pass the
IpAddress and Port number of the new node. This avoids having to query
the AD-SAL based properties set for this inforamtion

Change-Id: Ia2d8140c3f504d9b2d78b490bb273808f9a86c51
Signed-off-by: Dave Tucker <djt@redhat.com>

Deprecating the Schma-dependent Plugin CRUD APIs and Introducing Schema-independent CRUD API definitions in the Plugin layer.

1. Deprecated all the existing insert/get/update/Delete methods that are Open_vSwitch Schema-dependent
2. Introduced equivalent methods with DatabaseName as an additional argument.
3. Introduced a new inssertTree method that has the capability to insert a Tree of rows with parent-child relationship
4. Changed the return types to make more sense (instead of the status types returned today)
5. Added a more generic Exception class which will be inherited by more specific exceptions as the need araises.

The implementation for these methods are not done yet.

Change-Id: Ia363574a4d71e301f7c4b1a2e3bac08de53c62f9
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Refactor of the OVSDB Plugin

- Separate API and Implementation of services
- Deprecated existing interfaces
- Only export the OSGi services from the budle
- Remove ServiceHelper and use Dependency Injection
- Publish Ovsdb Connection events from the Library via OSGi

Change-Id: Ic9fac3747548a741998c40940f73244dae06626a
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge "Added LBaaS/OS API calls to the project Postma collection"

Merge "With the Controller Project exposing NSF features, we can enable OVSDB Northbound karaf features"

Merge "Bug 1471 : Fixing the Open_vSwitch Schema Controller Table's Target as String (from Set<String>)"

Added LBaaS/OS API calls to the project Postma collection

-Added LBaaS to the existing FWaaS and portSec.
-Made a directory for each service.

Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Merge "Init bundle files for ovs-sfc."

Init bundle files for ovs-sfc.

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

With the Controller Project exposing NSF features, we can enable OVSDB Northbound karaf features

Change-Id: Ibd9691b86d848029318c82e4c0870a42f5ce5462
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1471 : Fixing the Open_vSwitch Schema Controller Table's Target as String (from Set<String>)

Also added a IT test to cover this case.
This issue was uncovered while fixing https://git.opendaylight.org/gerrit/#/c/9606

Change-Id: Icf1b242f65a58aa6a38259ee0f7195d1ec946d7c
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Improve debug and trace logging in Neutron

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

Merge "NBI v3 Method Stubs"

Fixed OVSDB plugin Integration Test issue on Jenkins server

1. Removed SureFire plugin and made this FailSafe only IT
2. Moved IT files carrying individual test-case grouped under more logically placed IT classes
3. Changed the port name constants from eth0, eth1 to test0, test1 (somehow this affects Jenkins. Need to work on that more to understand the rootcause).
4. Moved all the Tests under TestSuite with TearDown

Change-Id: I8cc8225332910bcf0c56babb10010d55c9e8df99
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Include PAX Exam Code Coverage in Sonar

- Create an aggregate jacoco report in the root target direcorty
- Combine surefire and failsafe reports to all tests run in Sonar
- Split the Maven profiles for integrationtest and coverage
- Add a jenkins profilei
- Remove root IT profile so IT modules are complied every time

To run the integration tests

 mvn clean install
 mvn verify -Pintegrationtest

To run the IT with JaCoCo

 mvn verify -Pintegrationtest -Pcoverage

To upload results to Sonar

 mvn sonar:sonar

Change-Id: Ia0875de7efe67fc7a7d067adabed2d9588e284f4
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge "Fix behaviour of listeners in the InventoryService"

NBI v3 Method Stubs

This commit adds the methods stubs for the v3 NBI.
OVSDB can now meet the API freeze milestone while we work on the
implementation

Change-Id: Icdc78a64f31d4e62926284c4669ecec65877421a
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge "Added per-flow TEP and per-port TEP example in SW schema IT"

Updating an oudated README file

Change-Id: I9c6bc8b9699ecafb91421cc7a7cccb5d872e0b31
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Karaf features for OVSDB modules.

1. Individual Features for library, schema wrappers and plugin.
2. Northbound and Openstack features will be added as soon as controller AD-SAL karaf features are resolved.
3. Updated proper <name> tag in each of the pom files as the default karaf behaviour is to print artifact-id in bundle:list.

Sample logs after kar:install : https://gist.github.com/4845f27a79cf74dd7431

Change-Id: I2180c8dc37a4cc06d0ab9a53b03162594439ac85
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added per-flow TEP and per-port TEP example in SW schema IT

Added a TEP example into the sw schema.

Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Fix behaviour of listeners in the InventoryService

Currently a random OVSDB Inventory Listener will be resolved from the
OSGi Service Registry. This commit enables ALL registered inventory
listeners to receive updates using the OSGi Whiteboard Pattern

Additionally there was a hack in the Port and Network Handlers that
created InventoryService events, breaking the Provider/Consumer
contract. This is also fixed in this commit.

Fixes bug 1298

Change-Id: Iec1871a9022291b222a56cbb03e2093d3d45cc7a
Signed-off-by: Dave Tucker <djt@redhat.com>

Refactor OVSDB Neutron Code

- Any OSGi Service that implements the NetworkingProvider interface is
  available as a Neutron Networking Provider
- OFXProvider is now resolved based on the version on OVS installed on
  the bridge
- Bump version to 1.0.0
- Refactor main service classes and add Javadocs
- Adjust class naming to better reflect purpose
- Move constants and utils to specialized classes
- Split api and implementation
- Expose neutron services via OSGi
- Move providers to a seperate bundle
- Clean up Activator
- Remove ServiceHelper and instead rely on OSGi Service Registry
- Remove unnecessary OSGi Lifecycle Callbacks
- Use Guava to create new HashMaps and Lists
- Use Guava Preconditions in place of null checks
- Fix bug in OVSDB plugin where OF Controllers and Protocols where not
  set in OVS < 1.10.0
- Export the "lib.errors" package from the OVSDB library

Change-Id: If197f04b94acc8811ee925f5d0892792e13adbb6
Signed-off-by: Dave Tucker <djt@redhat.com>

Bug 1385 : Adding support for toString, equals and hashCode for the Typed Interfaces.

Guava's AbstractInvocationHandler indeed handles these missing methods on top of InvocationHander (that we currently use).
But unfortunately, the equals() support in AbstractInvocationHandler is screwed up and doesnt fit our needs.
More to it the invoke() method in AbstractInvocationHandler is declared as final.

Hence have implemented a similar functionality on top of InvocationHandler.

In order to have a proper hashCode and equals for these Typed Interfaces, I had to add these methods to all the
Library schema unTyped classes such as Row, Column, BaseType, etc...

Also added IT to cover these methods.

Change-Id: Ia5c3081ae5ce20b9a8fd6ceaadbfd21145649078
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1401 : Resolving unrecognized field "syntax" error in OperationResult

As per RFC 7047, the error field is defined only the the "error" and "details" elements.
But it can also contain other elements that is not described by the RFC.

      "An <error> MAY also have other members that describe the error in more detail.
      This document does not specify the names or values of these members."

The ovsdb-server implementation may choose to add more fields to give more information
to the error.

Since, these fields are not defined in the RFC, we should not fail the JSON parsing
if we encounter unknown fields.

Change-Id: I95c2e5ada6d8b97ff6dfe4f4ed9534ba143e822f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1406 : Resolving an  IndexOutOfBoundsException in Transact Response processing

Change-Id: I4bf94f9100147411c88e7bb862e9bb1c38127e3c
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Removing affinity from the ovsdb distribution to be in sync with integration virt edition.

Change-Id: I19e3594341b9ba76d15a2b8d5b1d113696ed7453
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Reorganize Pom Files

- Create better heirarchy
- Move plugin configuration to commons
- Use pluginManagement in parent instead of plugins
- Move integrationtest profile to parent pom
- Move ovsdb bundle versions to parent pom
- Add common dependencies to commons pom.xml

Change-Id: If5a5a668c17a9d4146bfdd3e7ab4b33938aaefb5
Signed-off-by: Dave Tucker <djt@redhat.com>

Use ODL Root Parent POM

Change-Id: Iefc44b02bfdca126d806df67ac3e18b60ceba5f9
Signed-off-by: Dave Tucker <djt@redhat.com>

Fix JaCoCo plugin to produce local reports

There seems to be a bug in JaCoCo where using "include" in the
configuration produces a blank report.

Change-Id: I34f97c2c27d5de569b8177736e0da920e87cdddc
Signed-off-by: Dave Tucker <djt@redhat.com>

Update .gitreview

Delete the defaultbranch as we are now on master

Change-Id: Ic9515c708884b97bd1ea062bcfc4d4e9eee1147d
Signed-off-by: Dave Tucker <djt@redhat.com>

Merge 'origin/topic/schema' branch into merge-branch

Change-set includes :

1. Schema Independent Library layer/bundle with no dependencies to other Opendaylight components
2. Library can be also be used in a non-osgi environment
3. Redesigned Plugin, Neutron and NorthboundV2 layers that make use of the Schema independent library
4. Static Typed Schemas - Open_vSwitch and hardware_vtep as a thin wrapper layer on top of library
5. Unit & Integration Tests with PAX-Exam integration for most of the newly added/modified code-base
6. Bundle names and versions are modified to reflect the new functionalities

Change-Id: I126de455692bc38f287ac2874da796f00f39ba7a
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Normalize Bundle Naming

Remove underscores and capitals from bundle names.
Remove duplicated "ovsdb" in bundle names.

This produces the following bundles:
- org.opendaylight.ovsdb.library-1.0.0-SNAPSHOT.jar
- org.opendaylight.ovsdb.neutron-0.7.0-SNAPSHOT.jar
- org.opendaylight.ovsdb.northbound-0.6.0-SNAPSHOT.jar
- org.opendaylight.ovsdb.plugin-1.0.0-SNAPSHOT.jar
- org.opendaylight.ovsdb.schema.hardwarevtep-1.0.0-SNAPSHOT.jar
- org.opendaylight.ovsdb.schema.openvswitch-1.0.0-SNAPSHOT.jar

This is consistent with the existing bundle naming in Nexus

This commit also fixes import issues with Eclipse...

Change-Id: Ie693bab44e9452d1f09c71181f3ccac87471fbbb
Signed-off-by: Dave Tucker <djt@redhat.com>

Fix incorrect OVSDB class names

OVSDB should be "Ovsdb" in Camel Case.
This patch fixes some mis-Camelizations

Change-Id: I70fa37e59ccfadbf9e28001cb18197e9c69dc19c
Signed-off-by: Dave Tucker <djt@redhat.com>

Fixing Activator to get InventoryService register with ConfigurationService properly.

Change-Id: Ic3785f2c74820208859015ce1b8f9663265869b0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Added new IT base for Hardware VTEP schema, Manager test case" into topic/schema

Added new IT base for Hardware VTEP schema, Manager test case

Change-Id: I9454a7f147b8d4f79cc9c15864f93530a1db4684
Signed-off-by: Matt Oswalt <matt@keepingitclassless.net>

OF10Provider missing one change to use ovsdbTable.getTypedRow

Looks like one instance of converting to the call was missed.

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

Merge "Use Long instead of Integer for ports." into topic/schema

Use Long instead of Integer for ports.

Add fixes in OF10Provider to use long for ports.

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

Backward compatibilty support for Northbound v2 GET operation.

Sample output for a GET operation based on latest topic/schema branch : https://gist.github.com/1fc3dd2eb4d2391d16df
This provides the backward compatible support as in Hydrogen release.

Northbound v3 support will bring in the new and improved version of GET/PUT/POST operation.

Change-Id: I082afe8ef87ad594f0467d96f103dbed45888d5f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Removed the static usage of open_vSwitch from transactBuilder and made it a dynamic value." into topic/schema

Merge "Removing unused classes from Library" into topic/schema

Removed the static usage of open_vSwitch from transactBuilder and made it a dynamic value.

The actual change is in TransactBuilder.java
-        List<Object> lists = Lists.newArrayList((Object)"Open_vSwitch");
+        List<Object> lists = Lists.newArrayList((Object)dbSchema.getName());

But this change of passing the databaseSchema to the TransactBuilder calls for changing a
few APIs across the library.

This fix is needed in order for hardware_vtep testing to proceed.

Change-Id: I3e923ab9edd8d0f6836704be0cb6bd209f5c0480
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Add more ignorables to gitignore

.vagrant should have been here but wasn't
.DS_Store is an OSX temp file that slipped in

Change-Id: I1436613929f850b0899a56cd7ec9a982c92cb31e
Signed-off-by: Dave Tucker <djt@redhat.com>

Removing unused classes from Library

Change-Id: I9397271ee485b0f04f64da1fb8582a9910160ba3
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1340 : Resolving TableUpdate handling that was not in complaince with RFC 7047.

1. Introduced RowUpdate class as per RFC 7047
2. Modified TableUpdate to use RowUpdate
3. Modified the parsing logic to make use of the new TableUpdate/RowUpdates
4. Modified the users of Monitor Updates to make use of the new TableUpdate
5. Removed an unnecessary redirection/ complexity in the NodeDB class
6. Adjusted and added support in the IT to handle multiple Rows in a single update.

Change-Id: I8c74d904e2c33087335825923830ef275cc818ac
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Add JaCoCo coverage for OVSDB" into topic/schema

Bug 1334 - Change the Typed Classes in Open_vSwitch and hardware_vtep schema

- Fixed IT tests.
- Address review comments: removed some commented out lines
- Modified schema files to use Set<Long> instead of Set<Integer>
- Modified user files to use new schema

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

Add JaCoCo coverage for OVSDB

- Update to JaCoCo 6.4
- Add UT and IT configurations
- Required to push UT/IT coverage to Sonar
- Can be run locally:
 mvn clean verify -Pintegrationtest
- Local coverage reports are in the target/site directory of each
  module

Change-Id: I5c4c1c4d2e7b37eb67f9bf2ade818c8ed4256cd0
Signed-off-by: Dave Tucker <djt@redhat.com>

Fixing a few issues in OF13Provider

Change-Id: Ib186a4e60aa1003e2e94d9b4298529250bb0f0e7
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Removed the executorservice thread that we added in async update processing

We moved channelRead in JsonRpc Layer to its own thread in order to avoid a possible channel-lock
issue when we try to program the table in the same thread.
But this breaks a lot of assumption in the existing code in Neutron/Library layers where an update
followed by access has to be delayed.

With the earler changes made to the Monitor request resulting in synchronous MonitorResponse, the
channel lock issue for the initial monitor processing is taken care of.

Any subsequent Updates are anyways thrown to an application queue which gets processed in its own
thread.

Hence, removing the thread from the low-layer and that enables receiving the monitor tableupdates/
updating the tableCache in the same thread.

Change-Id: I8a70b03fe87f8002bfc78a55b28d71c3e7152776
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Adjusting ovsdb project to work with the new default openflowplugin and openflowjava bundles." into topic/schema

Merge "Bug 1327 : Fixing a few Connection handling issues" into topic/schema

Adjusting ovsdb project to work with the new default openflowplugin and openflowjava bundles.

With the latest changes in the Controller and Integration projects, MD-SAL based Openflowplugin is made
the default openflow plugin. In order to work in tandem with Controller/Integration project, this changeset
contains the changes that will make MD-SAL Openflow Plugin as the default Openflow Provider in the OVSDB local
distribution target.

Change-Id: I11fc6bae0fb8c471fc8518baacf3e96eeefce1d1
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1327 : Fixing a few Connection handling issues

Since the library and plugin layers are seperated out into its own OSGi bundles and the initial low-level
OVSDB connection is managed by the library layer, there are cases in which the plugin layer might become
ACTIVE at a later point in time after the actual Passive OVSDB connection is established with the Library.
Also, the ConnectionManager and higher application layers can independently be restarted.

This fix brings in the robustness in the Connection management interactions between Library, Plugin, AD-SAL
and Connection Manager NSF.

In turn this makes the printCache command to work consistently which addresses the issue raised in Bug 1327

Change-Id: Ifc004f01e0306f7e71f908c063472a0b38bb572c
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Resolving possible NullPointerException during disconnect cases." into topic/schema

Merge "Fixing a ClassCastException issue with an incorect Row to TypedClass cast introduced during the migration work." into topic/schema

Resolving possible NullPointerException during disconnect cases.

During Connection Disconnect cases, the getTableName method can return null.
In such cases, an unprotected access to the return value can result in NullPointerException.
This fix addresses such a possibility.

Change-Id: Ia05c868dfd805af37f4a5fe9156d54582c753f1f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Fixing a ClassCastException issue with an incorect Row to TypedClass cast introduced during the migration work.

Change-Id: I8d60a4029db7986e852821148c3e6e1b8a619959
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Bug-1190: Fix for select response" into topic/schema

Bug-1190: Fix for select response

Updated transact interface in OvsdbRPC to return JSON response and
deserializing to OperationResult being handled in OvsdbClientImpl

Change-Id: I061292435e68a638f83e69495710ffaa3a04479b
Signed-off-by: Sudheendra Murthy <sudhi.vm@gmail.com>

Merge "Add ExceptionHandler to the Netty Channel Pipeline" into topic/schema