Merge "explicitly parsing JsonNode for updates" into topic/schema

explicitly parsing JsonNode for updates

Change-Id: Ib9f420c9f9d8eb11d459e3fc903b7280fa1431cb
Signed-off-by: Ashwin Raveendran <ashw7n@gmail.com>

Merge "Add unit test for JsonRpcDecoder" into topic/schema

Add unit test for JsonRpcDecoder

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

Removing the Open_vSwitch schema hardcoding in OvsDBClient.

Though it is an harmless constant string, it goes against the motivation of the library redesign to be
schema independent. Hence moving the schema string to a more appropriate place (to the applications),
in this case, to the Integration Test files.
Also, modified the initialize functionality to include a test to check for the list_dbs to contain Open_vSwitch
schema as all the tests in this file are based on Open_vSwitch schema tables.

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

Merge "Added Mutate operation" into topic/schema

Added Mutate operation

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

Added assert operation as per RFC 7047 section 5.2.10

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

Added Comment operation as per RFC7047 section 5.2.9

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

Fixed a bug introduced due to a recent commit. Thanks to Integration tests, we were able to identify the issue.

We need to add good Asserts to catch and Fail Verify and Merge jobs. That will prevent from these happening.
We are waiting on the Mutate operation to et merged before we could write better Asserts.

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

renamed old package to temp and fixes for min/max

Renamed the old package for TableUpdates to temp and have a newer one that
 can be enhanced for the current use caes while slowly deprecating the old
one. Also fixes a bug and added support for Set based field

Change-Id: If36c9c143eff2a8cc0cd1a0af5f1572ce1dbced5
Signed-off-by: Ashwin Raveendran <ashw7n@gmail.com>

Added a test-case to include non-atomic column to test the ser/de-ser pipeline in the new schema base code.

Please note that, this is added to make sure we will replace the ugliness of OvsDBSet/OvsDBMap from the Client
facing API towards internals.

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

Merge "Stop isReady method always returning true" into topic/schema

Stop isReady method always returning true

The OvsdbClientImpl.isReady() method will always return true as the
schema variable is initialized when the class is created and an
instance of a list can never be null.

This commit implements the timeout in the ToDo and changes the logic
to check if the schema.isEmpty

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

Merge "Add OVSDB Select Operation" into topic/schema

Add OVSDB Select Operation

Add the OVSDB Select Operation to the new OVSDB Library

Change-Id: Ie59033a7628f02e85bc3babba56c5723406398d0
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Update .gitreview to use topic/schema branch

`git review` will push to topic/schema by default.

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

Merge "Abort operation as per Section 5.2.8 of RFC 7047" into topic/schema

Merge changes Idd1bb88c,I9fd5ef99 into topic/schema

* changes:
  Fixed a bug in where condition usage in Update & Delete operations. Also moved Delete operation to a seperate transaction in the Test code to test multiple transactions.
  Commit operation as per Section 5.2.7 of RFC 7047

Merge "Cleaned up Integration Test setup" into topic/schema

Abort operation as per Section 5.2.8 of RFC 7047

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

Fixed a bug in where condition usage in Update & Delete operations.
Also moved Delete operation to a seperate transaction in the Test code to test multiple transactions.

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

Commit operation as per Section 5.2.7 of RFC 7047

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

Cleaned up Integration Test setup

1. Removed integration-test.properties from library and plugins
2. Removed unused json files from plugins.
3. Fixed all the places where some random default ipaddress / port is used
4. Forcing inclusion of ovsdbserver.ipaddress System Property when running integration Tests by an AssertFail.
5. Fixed a bug in OvsdbTestDeletePortIT

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

Replace literal for Update operation with constant

The update class uses a literal for the operation name.
Replace this with a constant to be consistent with other operations

Change-Id: I3512bedd5badf2b86e36619b859a6e0c53201761
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Add the OVSDB Delete Operation

Add the OVSDB Delete Operation to the new library

Change-Id: Ie16c193cca74184e79f1e8462dcc11bf8632a9d4
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Implementation for Monitor operation

Change-Id: I558099303c306d244a52fb2117e825c282609f3a
Signed-off-by: Ashwin Raveendran <ashw7n@gmail.com>

Code ReOrganization and Re-Architecture changes
1. Split OVSDB library from plugin
2. Removed all Opendaylight dependencies from OVSDB library
3. Modified the library to take in context vars to retain consistency with the current design.
4. Changed the artifact-id of all the bundles from random conventions to a consistent convention "ovsdb_*"
5. Also changed the directory structure to fit in the reorg/rearch.

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

Added more ignorable files to .gitignore

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

interface declaration for missing ovsdb operations
Fixing Merge-Conflicts manually as the Automatic Rebase failed.

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

Refactoring for better package organization and other changes.

The main interface is renamed to OvsDBClient to represent the library's
intention of being used against any OVS capable device. Fixed methods
to be all asynchronous.

Change-Id: I9d6e0c8cc53e0bcf188d572c655c7a28e3e0dae2
Signed-off-by: Ashwin Raveendran <ashw7n@gmail.com>

Merge "Add helper methods to meta/*Schema classes" into topic/schema

Add helper methods to meta/*Schema classes

Add the following helper methods to make error and type checking easier.

DatabaseSchema.getTables() - gets a List<String> of tables
DatabaseSchema.hasTable(String table) - returns true if the table exists
DatabaseSchema.getTable(String table) - returns the TableSchema object
TableSchema.getColumns() - gets a List<String> of columns
TableSchema.hasColumn(String column) - returns true if the column exists
TableSchema.getColumn(String column) - returns the ColumnSchema object
TableSchema.getColumnType(String column) - returns the ColumnType object
for the specified column

These calls are intended to be used for error/type checks. For example:

if (!device.hasTable("Open_vSwitch") {
    throw RuntimeException("This is not the vswitchd schema")
}

Change-Id: If9dd286c05a38d7f60a0ce63cd3148157e3b345e
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Add license info to new files

Change-Id: I175d777952cac6109dfd838d3938446f241ea8e6
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Bump up the bundle versions for the bundles based on Semantic Versioning.

Since we are performing major rearchitecture of the base OVSDB library and it results in backward incompatible
API changes, the MAJOR version is bumped up. But the neutron and northbound bundles will face functionality impact
but not API impact & hence only the MINOR version is bumped up.

Care must be taken to make sure the master and topic/schema branche versions dont step on each oth versions.
It causes CI headaches.

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

Fixing checkstyle issues on Ashwin's initial POC code for the new Schema design.

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

Merge "Adding support for the git-review command line tool"

Updating ovsdb dependencies to the LATEST SNAPSHOT in Controller & Yangtools project

After the Hydrogen Release, we decided to keep the inter-project dependencies only with the RELEASE version of the bundles.
But, Since the weekly release schedule is not ratified and actual SNAPSHOT dependencies are growing, we dont have much choice
other than to depend on the SNAPSHOT versions of the bundles.

Once we have a streamlined process for weekly RELEASE in place, we can change our strategy to depend only on the RELEASE bundles.
Till then, we have to depend on SNAPSHOTs to make progress.

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

Update Enunciate Documentation for OVSDB REST API

Change-Id: Ia77888f75343075710de2c1522cbd048bd609543
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Removed convertNeutronIDToKey(String) calls.

The convertNeutronIDToKey(String) method of BaseHandler class converts
Neutron ID.
Because PortHandler class just uses Neutron ID for log output, it does
not need to use convertNeutronIDToKey(String).

Change-Id: I025b1db5cdc0c635a75a286f6eeb961c5233f7fa
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

Fixed typo in PortHandler.java.

Change-Id: I85204b6abccba995caf8258db2227d2c488713c7
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release master-tagforprepareonly-ovsdb-bulk-release-2-1

Adding commons/parent to the root pom.xml <module>

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

Fixed trailing / to satisfy the versions plugin

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

Update to new version of the artifacts proposed by jenkins-ovsdb-version-changes-8

Change-Id: If628434027c91a82c6b4a72baf04959f5a2ac4a0
Signed-off-by: Jenkins ovsdb silo <jenkins-ovsdb@jenkins002.cvo.opendaylight.org>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Removed the optimization that was preventing the flows to be programmed on nodes where the Network wasnt present.
But, since we dont have the replay fix which can look back at all the nodes and reinstall the existing VM Macs,
we have to remove this optimziation.

We can reintroduce this fix, once we have the replay functionality in place.

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

Merge "Upgrading Yangtools to 0.6.1-SNAPSHOT and hopes it wont cause any runtime issues"

Making the FLowMods Strict. During update operations, a Non-Strict flow is being incorrectly updated by other flowmods.

With this fix, we are able to get DevStack working with ODL-OVSDB-OF1.3 combination.
Hopefully this is the final functional fix before the Hydrogen release.
Definitely a lot of improvements can be made. But we have a good working start to this story.

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

Upgrading Yangtools to 0.6.1-SNAPSHOT and hopes it wont cause any runtime issues

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

Moving Tunnel Out to the appropriate place and cleaned up some logs.

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

Fixed a class cast exception when the Node-ID is a Long that got casted to String.

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

Moved InstructionBuilder order and key in OF13ProviderManager
to the OVSDB tunnel implementation.

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

Merge "Fixed the Repositories and PluginRepositories directives to include nexus group repo."

Fixed the Repositories and PluginRepositories directives to include nexus group repo.

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

Change OF13_Provider to ovsdb.of.version in config.ini

- Defaults to "1.0" when no provider is set in config.ini
- Values other than "1.0" and "1.3" will get the OF10Provider
- Case statement added so providers for other versions can be esaily inserted

Change-Id: I878f594ce2b84c5809c7a8077e04fbb76c2fece8
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Internal Vlan deletion and Tunnel Deletion cleanup

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

Remove call to networkCreated in NetworkHandler
Tenant VLAN assignment is handled as part of the interface update.
This now means that each node does not need to allocate a VLAN for each tenant.

Change-Id: If817e54a603e2abb3312b9c98a0104dee128cfb2
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

OVSDB Log Cleanup

Change-Id: I811c28b4f45d7eb5fa8656b65381059b1928a911
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Fix ConnectionService to remove stale connections from the ovsdbConnections list

Change-Id: Ie6c23aab01858aeebc2bbd4b6a76a1993ed8d91d
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

DROP and FLOOD/Broadcast Support

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

Added fix to prevent re-adding tunnel that was just deleted.

Change-Id: I0b2275d986dfb5a93fb22794634caa130b4fa795
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>

Replace tenantVlanMap with a per Node cache

Change-Id: I8c2aeb9960443bd9d7e51315f69f499631c91b69
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Resolved the OF10 to OF13 transition issues during devstack bootup.

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

Release root pom should have a relative path

- This should be able to fix the wrong tagging calculation

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

Fix for bug 278 Replace implementation of dbCache in InventoryService with a ConcurrentHashMap Replace implementation of cache in NodeDB with a ConcurrentHashMap

Change-Id: I95549730117d5ce330524cb63098e2e931c61a29
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Fixing a pom file issue that fails the dry-run

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

Fix to use MacAddress for Mask for Ethernet Matches

Goes with:
https://git.opendaylight.org/gerrit/#/c/4638/ (openflowplugin)
https://git.opendaylight.org/gerrit/#/c/4635/ (controller)

Change-Id: I10303d6c1dd3bf2ac60d1383e5645ce341fb2390
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Reorganized the Local port programming to support the missing single node case & All the local forwarding.

Also, moved the Flow-IDs from a static value to a unique id per flow.

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

Merge "Removed the tunnelEndpoint cache Tunnel Endpoint address is now pulled directly from the Open_vSwitch table"

Removed the tunnelEndpoint cache
Tunnel Endpoint address is now pulled directly from the Open_vSwitch table

Change-Id: I1a7292d2e4921c3d9461c79112a4a68d5fb5f33a
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Removed printstacktrace method call from ProviderManager.

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

Set the default fail-mode as secure and make sure to set the Openflow Type appropriately on both insert and update case.

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

OVSDB Neutron & OpenFlow 1.3 integration

This commit brings in a major portion of OVSDB Neutron & Openflow1.3 integration pieces & is
built on top of Brent's OF1.3 work.

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

Merge "Fixed a typo in pom.xml s/includeArtifacIds/includeArtifactIds"

Fixed a typo in pom.xml s/includeArtifacIds/includeArtifactIds

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

OVSDB support for controller restart scenario to work along with Neutron interface.

With the Controller support for neutron config save/restore in, this fix handles the next level
of bootup handling in the ovsdb's neutron handling.
Essentially, the southboundhandler triggers the update for the existing OVSDB connection and that
handles all the programming needs.

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

Merge "Added OF v1.3 flowmod builders for neutron integration."

Added OF v1.3 flowmod builders for neutron integration.

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

Fixed the OVSDB Northbound API Postman Script that works with the Jackson Upgrade

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

Deleted the POSTMAN script that got invalid after the jackson upgrade.

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

Merge "Refactored IT's to remove dependency on com.google.code.gson.internal.Pair Renamed IT's to fix typos and keep case consistent"

Make automagically adding a controller to each detected bridge after connecting an OVS node a configurable setting. The default is ovsdb.autoconfigurecontroller=true

I've seen multiple configurations where the ip address of a host is configured on the bridge interface.
So for example there is one bridge (cloudbr0) and one interface (eth0). Eth0 would be added as a port to cloudbr0 and the ip address of the host would be configured on cloudbr0. This is actually the recommended configuration when using CloudStack with a Xen or KVM hypervisor running OVS. I've seen this configuration is multiple other setups as well.
When a node with this configuration is added to the ODL controller, all communication will be blocked as a controller implies that the switch will be flow based. This causes ovsdb to lose communication to the outside world and effectively isolates the hypervisor from the network.

This commit adds a property to configure this feature from config.ini. Set "ovsdb.autoconfigurecontroller" to true to configure controllers on all detected openvswitch bridges.

Change-Id: I8285bd2b8a7000997b4b62373bd9e02457c93d8e
Signed-off-by: Hugo Trippaers <hugo@trippaers.nl>

Refactored IT's to remove dependency on com.google.code.gson.internal.Pair
Renamed IT's to fix typos and keep case consistent

Change-Id: I97bf39df8a8a6b127309a495e1a8b58cba92d291
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Harmonize the use of <scm> in pom.xml files

Modify pom.xml files to have the same <scm> section.  Fix typo in some
of the <developerConnection> URLs.

Change-Id: I761cbc67d5230470501eacd8f474ec113c8a73d7
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Update to new version of the artifacts proposed by jenkins-ovsdb-version-changes-1

Change-Id: I89d7c9944b76265650ebbc1839e1cf012b63ec77
Signed-off-by: Jenkins ovsdb silo <jenkins-ovsdb@daylight7.linux-foundation.org>

Fix build error, due to changes in FlowId: https://git.opendaylight.org/gerrit/#/c/4058/

Changes to fix build were based on changes done in commit 45b75a6ba76bf8b453b5f0de11ceb0c3bba1e842, file:
.../opendaylight/md-sal/statistics-manager/src/main/java/org/opendaylight/controller/md/statistics/manager/StatisticsUpdateCommiter.java

Change-Id: I635c7530cd9897d7c0ee8f1cb5209869c90f24b5
Signed-off-by: Flavio Fernandes <ff@member.org>

Merge "Fixing the ColumnSchema as per RFC7047 for a enum variable."

Fixing the ColumnSchema as per RFC7047 for a enum variable.

Enums are defined as a 1 or more atomic-type and hence the current Object[] is incorrect
and is causing deser issues with vtep schema. Fixing it with a more generic Object.
This will be properly accomodated when we have schema driven OVSDB approach is in place.

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

Move the java 1.7 config to the parent pom into pluginManagement so it is enforced for neutron and northbound as well.
Merge failed due to conflicts and hence manually applying Hugo's changes on his behalf.

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

Updating the commons parent pom to 1.4.1-SNAPSHOT and checkstyle fixes

ovsdb project depends on the controller project's 1.4.0-SNAPSHOT version of commons pom.xml.
Since the latest changes (including the jackson upgrade) is done on 1.4.1-SNAPSHOT, we have
to update to the latest.
While doing so, it caught quite a few checkstyle errors, which am fixing it along.

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

Merge "Added Mockito and PowerMock to the neutron pom.xml Unit test coverage for AdminConfigManager Unit test coverage for NetworkHandler"

Merge "Prevent NPE when an interface is already present on the switch, but no related port is configured."

Merge "Small Java related improvements."

Prevent NPE when an interface is already present on the switch, but no related port is configured.

Change-Id: I966b412d06450f75ea8940dc413b9973ec1fc1d1
Signed-off-by: Hugo Trippaers <hugo@trippaers.nl>

Aligning Jackson to the Controller's l latest upgrade (2.3.0). Also updated the northbound bundle to use the latest Jackson.

Due to this change, the ovsdb.northbound bundle will start to accept and respond OVSDB JSON messages in the same format as
expected by the ovsdb-server. (i.e) a set will be represented as ["set",[]] instead of the common sense [].
This is due to the ovsdb requirement as specified in the RFC / Draft.

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

Small Java related improvements.

Remove redundant return statement from void method,
remove redundant boxing of numeric variables,
remove statements that check a condition that is always 'true',
add braces to 'if else' statements.

Signed-off-by: Alissa Bonas <abonas@redhat.com>

Adding support for the git-review command line tool

The OpenStack doc workflow (and entire gerrit workflow) has the
option of using a tool called git-review that automates ChangeID:
creation and management,automates rebasing of local commits
as well as simplifies submiting to gerrit and pulling and
pushing reviews.
Usage of the tool is entirely optional, however it does require
a pointer file named .gitreview in the root of the repository
that is referenced.
I am totally new to this project, if you guys don't want to merge
this then no big deal. However I think this will help with dev's
coming over from OpenStack as it will provide a optional workflow
that is closer to the OpenStack workflow.
Necessary to run git review -s to append the change-id into git-hooks

Change-Id: Id8ea7edefe0291c0f22db8c3ef910776708fac30
Signed-off-by: Colin McNamara <colin@2cups.com>

Added Mockito and PowerMock to the neutron pom.xml
Unit test coverage for AdminConfigManager
Unit test coverage for NetworkHandler

Change-Id: Ice7582e2a9f4dd5a4b7c9fdac01e86196f48ccf4
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

OVSDB project's OpenFlow 1.3 ProtocolPlugin integration via Controller's MD-SAL infrastructure

This checkin brings in the basic infrastructure changes needed to integrate the ovsdb.neutron bundle
with OpenFlow1.3 via the Controller's MDSAL infrastructure. The actual work of mapping the multi-table
and various Flow programming can be done now over this infra.

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