Add Interface Table Columns and Version Info

Also adds IT for Port and Interface tables
Additionally fixes camel case errors in OVS Table

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

Adding generic GETDATA and GETCOLUMN proxy methods for _uuid and _version in TypedBaseTable and enhanced the IT to monitor on OpenVSwitch Table

Please note that as per RFC 7047, the _uuid column will not be included for filter-less monitors and hence getUuid will not work if we are monitoring with no Filters.
But the TableUpdate contains the UUID for each of the update and a monitoring solution should manage that.
Please refer to TypedVSwitchdSchemaIT.java 's getOpenVSwitchTableUuid() for more info on how this is used.

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

Added Tests to test the Typed Class helper routine getTypedRowWrapper() and the Proxy getters with the help of monitor/update messages.

The tests are incrementally built on top of the existing Bridge create and Controller update test-cases.

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

Fixed UUID deserialization issue

As per RFC7047, UUID notation is defined as :

   <uuid>
      A 2-element JSON array that represents a UUID.  The first element
      of the array must be the string "uuid", and the second element
      must be a 36-character string giving the UUID in the format
      described by RFC 4122 [RFC4122].  For example, the following
      <uuid> represents the UUID 550e8400-e29b-41d4-a716-446655440000:
      ["uuid", "550e8400-e29b-41d4-a716-446655440000"]

But our current parsing in BaseType and ColumnType didnt take this into account.
This fix addresses the issue.

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

Merge changes I6244de55,I20d073c2 into topic/schema

* changes:
  IT for Typed schema Table : Controller
  Removed unneccessary usage of Maps.newHashMap from the IT files

Merge "Simplifying the User facing APIs in TyperUtils." into topic/schema

IT for Typed schema Table : Controller

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

Removed unneccessary usage of Maps.newHashMap from the IT files

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

Simplifying the User facing APIs in TyperUtils.

Based on Ashwin's comment to the library infra gerrit (8104), more simplified User Facing APIs are introduced :
1. Bridge rBridge = ovs.createTypedRowWrapper(TestBridge.class);
   replaces
   TestBridge rBridge = TyperUtils.getTypedRowWrapper(dbSchema, TestBridge.class, new Row<GenericTableSchema>());

2. rBridge.getSchema()
   replaces
   GenericTableSchema rBridgeSchema = TyperUtils.getTableSchema(dbSchema, TestBridge.class);

More such Simplified alternative APIs will be added based on usage.

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

Merge "Add version information and columns to Port Table" into topic/schema

Add version information and columns to Port Table

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

Merge "Add version information to the Bridge table" into topic/schema

Add version information to the Bridge table

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

Add version information to Open_vSwitch table

- Fix typo in `cur_cfg`
- Remove `status` as it is not a valid column

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

Add fromVersion and untilVersion to Typed Annotations

This allows a column or row to be validated against a specific version
of the schema. This gives better error messages to the user than a
simply asserting that a Table or Column does not exist

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

Merge "Adding rest of the Open_vSwitch schema typed tables." into topic/schema

Merge "Introducing Open_vSwitch Typed Schema wrapper on top of the Schema indepedent Library." into topic/schema

Add schema version to DatabaseSchema

This commit also adds a Version class to lib.notation that allows
versions to be compared. This is necessary for adding fromVersion
and toVersion to the TypedTable and TypedClass annotations

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

Adding rest of the Open_vSwitch schema typed tables.

Please note that these contain exactly the same column definitions as in the existing plugin.
We need to spend time filling up all the missing columns.

Also IT is not included for these Typed Interfaces. They will follow soon.

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

Introducing Open_vSwitch Typed Schema wrapper on top of the Schema indepedent Library.

Adding Bridge & OpenVSwitch Tables along with IT for these 2 tables.
These tables are refactored from the old plugin and hence not all Columns are statically Typed yet.
Those will be added gradually based on usage pattern.

More OpenVSwitch tables will be added soon.

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

Added proper Typed Schema Wrapper infra to Library which enables a simple mechanism to introduce any Schema specific Java Objects.

* Added ovsdb.lib.schema.typed package with infra classes that supports this functionality.
* TyperUtils.java uses Reflection to provide the required Wrapper functionality with the help of Java annotations (TypedTable & TypedColumn)
* MethodType option of TypedColumn is modified to support GETCOLUMN in addition to GETDATA and SETDATA method types
* Changed the existing Typed IT code and introduced TestBridge.java (Bridge Table of ovs-vswitchd schema)

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

Merge remote-tracking branch 'origin/master' into merge-branch

Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Conflicts:
.gitignore
distribution/opendaylight/pom.xml
neutron/pom.xml
neutron/src/main/java/org/opendaylight/ovsdb/neutron/InternalNetworkManager.java
neutron/src/main/java/org/opendaylight/ovsdb/neutron/SouthboundHandler.java
neutron/src/main/java/org/opendaylight/ovsdb/neutron/TenantNetworkManager.java
neutron/src/main/java/org/opendaylight/ovsdb/neutron/provider/OF10Provider.java
neutron/src/main/java/org/opendaylight/ovsdb/neutron/provider/OF13ProviderManager.java
neutron/src/main/java/org/opendaylight/ovsdb/neutron/provider/ProviderNetworkManager.java
northbound/pom.xml
ovsdb/pom.xml
ovsdb/src/test/java/org/opendaylight/ovsdb/lib/message/OVSDBNettyFactoryIT.java
ovsdb/src/test/java/org/opendaylight/ovsdb/plugin/OvsdbTestDeletePortIT.java
plugin/src/main/java/org/opendaylight/ovsdb/plugin/ConfigurationService.java
plugin/src/test/java/org/opendaylight/ovsdb/plugin/OvsdbTestBase.java

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

Removed the ovsdb subdirectory properly in lieu of the upcoming rebase with master

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

Fixed the broken Typed IT tests & included it to run as part of failsafe maven plugin.

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

Moved the code to normalize the java objects (Set and Map) to OVSDB json friendly format into ColumnSchema class

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

Added safety check for a case in which getSchema may not report a "hidden" Column, but Monitor respond with data for that "hidden" column.

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

Adding support for Unfiltered Monitoring

RFC 7047 supports unfiltered monitoring, wherein an application can send a Table Monitor request without specifying any
Columns to filter upon. OVSDB server will respond with TableUpdate for the requested table with data from all of its
columns (except _uuid).
In order to add this support, changes were made to MonitorRequest with an option of keeping its "columns" member as null.
Jackson serialization will ignore any member with a null value which will aid in the unfiltered monitor request.

Tests were added to cover this case as well.
Also thanks to this IT test, an existing bug is unearthed wherein the special column name was "_version" instead of "version".

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

Fixed Mutate operation to handle MultiValued {Key, Value} ColumnType.

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

Resolved the Serialization and Deserialization issues for MultiValued Columns with the specialized "set" and "map" indicators.

Fixed various classes where the KeyValue ColumnType is not handled.
Also added IT tests to cover both Serialization and Deserialization aspects of KeyValue ColumntType by using the external_ids column of Bridge table.

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

Modified the Library to be more friendlier towards the uuid value obtained through Table updates and the _uuid column.

This change involves the following :

- Populated columnschema for internally generated columns(_uuid and version) by default to the TableSchema (Refer RFC 7047 section 3.2)
- Added "uuid" field in TableUpdate, without which the applications were unable to get the Row identifier
- Moved the generic Object in OperationResult to Row<GenericTableSchema>. But this change alone is not enough to make Select operation
  work as expected. We need a more richer way to populate the "columns" in Row class during the Select response (Bug 1190)
- Modified OvsDBClientTestIT.java to use the uuid column obtained from open_vSwitch monitor in order to insert and delete Bridges during
  add and delete operation (Note that the current IT code updates the open_vSwitch row without this where clause).

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

Addressing review comments for 7920
Changes include :
- Changes variable name from tval to untypedValue in Insert and Mutate classes
- Fixed most of the review comments in the IT

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

Fixed the Basic Bridge create Integration Test by performing mutate on OpenVSwitch Table.

Also added
- Fixed a multiValue bug in Mutate
- the cleanup @After method to delete the created br-test
- few global variables that can be reused across test and cleanup (such as BridgeUuid)
- Fixed a Compiler warning in Insert

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

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

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

explicitly parsing JsonNode for updates

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

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>

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>

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>

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>

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>

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>

Merge "Add unit tests for getPhysicalInterfaceName"

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>

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>

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>

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 "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>

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>

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>

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>

Implementation for Monitor operation

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

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

Change-Id: Ic3620230e29d278d234a3ba9fea53da8f6ab5c8c
Signed-off-by: Flavio Fernandes <ffernand@redhat.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>

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>