Add missing check for status when doing programTunnelRules after addTunnelPort

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

BUG-1188: Missing callback to provider

When SouthboundHandler::processRowUpdate() gets invoked for a port,
the code was missing a callback to handleInterfaceUpdate() which
should be needed for all the interfaces for the given port.

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

Merge "Adding Pax-Exam infra with a basic IT for plugin"

Adding Pax-Exam infra with a basic IT for plugin

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

BUG-1147: Fix programming of NORMAL flow entry when using OF 1.3 Provider.

The OVSDB Plugin programs the br-int and br-ex bridges with the appropriate
flows as they are discovered by the plugin. The OpenFlow 1.0 provider
correctly installs the LLDP and NORMAL flows, but the OpenFlow 1.3 provider
does not install the NORMAL flow on the br-ex bridge (note: the NORMAL flows
are only installed on br-ex bridges, and not br-int bridges, when using
OpenFlow 1.3).

This patch fixes this by programming of the NORMAL flow on br-ex
bridges when using the OpenFlow 1.3 provider.`

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

BugFix #958 in OVSDB setting LLDP length to 0xffff

Patch4: rebase

Patch3: Setting the max length to the max value
to prevent truncation of the packets. The default
contrustor was (0). Packets are no longer being
truncated now.

-For the commentors here is the output of removing
maxlength():
-----------
OFPT_PACKET_IN (OF1.3) (xid=0x0): total_len=89
in_port=1 tun_src=172.16.86.129
tun_dst=172.16.86.128 (via action) data_len=0
(unbuffered) (***total_len != data_len***)

-With output.setMaxLength(0xffff); at max val
output is as follows:
--------------------
OFPT_PACKET_IN (OF1.3) (xid=0x0): total_len=90
in_port=LOCAL (via no_match) data_len=90
buffer=0x00000101

Patch2:
Pulling the max_len all together should solve
the truncating issue. Hard to tell as only half of
the flowmods are getting installed at the moment
and this creates lots of deserialization errors
from some IPv6 punts. It doesnt solve the issues
in the Neutron plugin at the moment. The original
error doesnt appear for me outside of OVS logs. There
isnt a need to send any packet payload which is what
the max_length field is for per the spec. I was thining
zero would send the whole payload but per the spec
and thought ofprovider had a good reason to use 40bits
but not so much. Will add comments there.

/*
From of1.3 spec section A.2.5
"The max_len indicates the maximum amount of data from
 a packet that should be sent when the port is
OFPP_CONTROLLER. If max_len is zero, the switch must
send zero bytes of the packet. A max_len of
OFPCML_NO_BUFFER means that the complete packet
should be sent, and it should not be buffered."
*/
Change-Id: Ifcbe22e38d599096a2e451144b461e128a41ee08
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Merge "Adding neutron api v2.0 security group/rules handlers"

Adding neutron api v2.0 security group/rules handlers

Patch3:
-Verified with merged patches and rolled back
whitespace formatting on Activator and Tenantmanager.

-Removed the comments Sam wanted removed.

-Only dependancy is commit 7410 that registers
the service in networkconfig.neutron.
https://git.opendaylight.org/gerrit/#/c/7410/

Patch2: Pulled the TenantManager adds since it was just
for a usage preview. Added NeutronPortSecurityHandler
parent handler for NeutronSecurityGroup and
NeutronSecurityRule. Still depends on NBAPI merges and
OF merges.

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

Sample OpenStack/Neutron v2.0 Postman API Collection

-Example v2.0 NBAPI Neutron calls to import into Postman
for development testing. All calls are either implemented
or pending merges.

-OpenStack Neutron v2.0 API reference:
https://wiki.openstack.org/wiki/Neutron/APIv2-specification

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

Merge "Add unit tests for getPhysicalInterfaceName"

Merge "getPhysicalInterfaceName not finding match in some cases"

Add unit tests for getPhysicalInterfaceName

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

getPhysicalInterfaceName not finding match in some cases

Cases where there are multiple Open_vSwitch rows would cause
the method to not find a match. The method should loop through
all the rows until a match is found, but an early break was causing
the method to exit after the first row is processed.

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

ovsdb has a dependency on artifacts from integration project, that shall be removed

- ovsdb has a dependency on artifact from integration project, while
integration project depends on ovsdb. This is a circular dependency
for the release process because we cannot release OVSDB without
releasing INTEGRATION and viceversa. To break the circular dependency
i'm proposing to build the ovsdb distribution starting from the
controller one adding the few deltas needed. Yes this cause some
duplication but i couldn't find a better way to break it. Suggestions
are more then welcome!

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

Merge "Fix bug 1028: Don't ignore exception in prepareInternalNetwork"

Add ovsdb vlan support by using OF1.3

This change is to support vlan tenant isolation using OF1.3
This is using OF1.3 and Sam has the 1.0 change.

The changes of common neutron files are committed by Sam.
Most of the change here is for OF1.3 only.

This commit also has dependency on BUG 1020 "Multiple output actions is not allowed in the action list of apply-action instruction". The vlan and tunnel broadcast is broken due to BUG1020.
We don't want BUG 1020 to block the vlan feature commit and send this change for code review now.
Onve BUG 1020 is fixed, broadcast should work properly.

Change-Id: Ibd16404212cc1c4de21921477e4d1cf335a12b39
Signed-off-by: Hsin-Yi Shen <hshen@redhat.com>

Use Felix DM for OVSDB Neutron Services

The ovsdb.neutron bundle has serveral services currently implemented as
singletons. This makes these classes difficult to test and also creates
a hard coded dependency. We can use the Felix DM provided by the OSGi
container to implement dependency injection for these services.
Fixes bug 1030

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

Fix bug 1028: Don't ignore exception in prepareInternalNetwork

As detailed in bug 1028, don't ignore the exception from
createIntegrationBridge as calling initializeFlowRules will fail.

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

Script support for DevStack Configuration.

Addition of several scripts that are helpful for DevStack
environment setup.

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

Moving ipfix column to set<UUID> in Bridge table.

Please note that these fixes to the fixed tables will all be deprecated once the topic/schema
branch is collapsed into the master. These fixes are encouraged into the master only to support
existing users. That is the only reason we are not spending time with UT/IT scripts for these.

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

Remove com.google.inject from neutron/pom.xml

This sneaked in when I was testing out com.google.inject.
Unfortunately it causes errors when starting OSGi. Ooops!

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

Merge "Add Vagrantfile for OVSDB Test/Dev environment"

Merge "Add License to OVSDB Repo"

Update OVSDB Pom files for sortpom plugin

Auto-sorting of pom files was introduced in
https://git.opendaylight.org/gerrit/#/c/6247/

As the OVSDB pom's are children they too must be auto-sorted.
`mvn clean install` was used to sort the poms for this commit.

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

Add License to OVSDB Repo

OVSDB doesn't have the full text of the EPL in the base of the repo.
Other projects (controller, vtn etc...) do. This commit adds the
license.

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

Add Vagrantfile for OVSDB Test/Dev environment

This commit adds a Vagrantfile to the root of the repository. This
allows a developer to issue a single command to pull up a development
and test environment:

    vagrant up

This will create 3 x VMs in Virtual Box. VMs can be
created individually if required.

1 x VM for Mininet - 2GB RAM
1 x DevStack Control/Compute node - 4GB RAM
1 x DevStack Compute node) - 4GB RAM

See README.Vagrant for full instructions

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

Merge "Fix unhandled timeout in OVSDBNettyFactoryIT"

Fix unhandled timeout in OVSDBNettyFactoryIT

The IT sometimes hangs when run due to a timeout in getting the schema.
Limit connection attemtps to 3 seconds with 3 retries.
This does not fix the root cause, but *does* fail the IT.

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

Merge "Update netty to 4.0.17"

Replace String Literals in ITs with constants

Code cleanup.
Replace String literals in Integration Tests with contants.

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

Fix failure of OvsdbTestGetBridgeDomainsIT

OvsdbTestGetBridgeDomainsIT was failing due to a NullPointer to the
ConfigurationService.invetoryServiceInternal

- Add InvetoryService to TestObjects
- Call setInventoryServiceInternal on the ConfigurationService

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

Fix integration tests

Fix hardcoded values and connection issue in OVSDB IT

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

Fixed distribution build parameters to pull in the Virtualization edition from Integration branch
and replace it with the latest OVSDB bundles (ovsdb library, northbound and neutron).

This makes the OVSDB developers life simpler and avoids all the copy the bundle over issues and
provides a much simpler way to execute the controller with the OVSDB bundles being developed.
Run the controller normally :

run.sh -Xmx2048m -XX:MaxPermSize=512m -virt ovsdb [-of13]

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

Removed all the static string usage for network types vxlan, gre, vlan and replaced it with a constant.

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

Add support for vlan tenant isolation to ovsdb using OpenFlow 1.0.

This is an initial push of the changes needed to support vlan tenant isolation.
Ignore the logger's with shague in them since they are for my debugging and will go away.

This is using OF 1.0. Hsin-Yi has the 1.3 task.

The br-eth for vlan and br-tun for tunnels has been combined to br-net. A single patch port
connects br-net to br-int.

Functions to parse the physical interface name from the provider_mappings have been added.

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

BUG-954: Add support for Neutron port/network delete events for OF1.3

This commit will enable support for Neutron port delete and network delete events.
It will also enable flow cleanup and tunnel port delete for ovsdb using OF 1.3

Signed-off-by: Hsin-Yi Shen <hshen@redhat.com>
Change-Id: Ic848125d21abc44c796ef2b1675cca9e50630f76

Update netty to 4.0.17

This is a follow-up for https://git.opendaylight.org/gerrit/#/c/5630/ .

Signed-off-by: Robert Varga <rovarga@cisco.com>

Turn hard coded localhost:8080 into templates in the postman_collection files.

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

Patched some existing and added IPFIX to Postman API Collection.

Changes:
-------
Added:
IPFIX API Calls:

Modified/Patched:
SFlow
Queue
NetFlow
Controller
Manager

For version tracking I opened Bugzilla ID 667
https://bugs.opendaylight.org/show_bug.cgi?id=667

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

Merge "Add build script in OVSDB resources"

Fixing a few issues in PatchSet-1 that adds support for IPFIX table in ConfigurationService.

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

Add build script in OVSDB resources

The script in this commit simplifies the build process of the OVSDB
project.
usage: ovsdb-build [-h] [-r] [-b] [-of13] [-v] [-g CHANGE_ID] [-start]
[-stop]
                   [-d DIRECTORY] [-t]
optional arguments:
  -h, --help            show this help message and exit
  -r, --reclone         Reclone the Controller/OVSDB repos
  -b, --rebuild         Build the Controller Project
  -of13, --openflow13   Enable OpenFlow 1.3 Support
  -v, --verbose         Verbose output
  -g CHANGE_ID, --gerrit CHANGE_ID
                        Gerrit Change ID
  -start, --start       Start the controller
  -stop, --stop         Stop the controller
  -d DIRECTORY, --directory DIRECTORY
                        directory
  -t, --tmux            Create tmux session

On first run it is recommended to run the following:

./ovsdb-build -r -b -start

You can add the `-d` option to specify the directory you want to build
in:

./ovsdb-build -r -b -d /home/dave/dev/opendaylight -start

This will clone the controller and ovsdb projects, build them, move the
ovsdb artifacts in to the controller directory and start the controller.

Gerrit Changes can be built and loaded to the controller with a single
command:

./ovsbd-build -g 5716 -start

The controller is started in the background. To access the OSGi ole
you can manually `telnet 127.0.0.1 2400` or you can add `-t` to let
the build script create a tmux session for you.

Change-Id: Ie98942843a90193a38e17e4a8cd09b2806942e5c
Signed-off-by: Dave Tucker <dave.j.tucker@hp.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."