Merge "Return only the Identifier and not the whole Node object"

Merge "Added support for fail-mode."

Return only the Identifier and not the whole Node object

The northbound should only return a string like "OVS|192.168.120.1:6640" when returning Node information. In the migration work the mdsal Node.fromString() returns all the Node object information and not just teh identifier as the adsal Node did.

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

Added support for fail-mode.

Supports both write of fail-mode to config, and also
read of it from operational data-store.

Also updated postman collection

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

Minor: minor edits and remove deprecated call for InstanceIdentifierBuilder

- Make code more readable by removing unecessary class specifiers
- use .build() instead of InstanceIdentifierBuilder.toInstance()

Patch 4, code review:
- move instance creation of flow and node to common functions

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

Postman collection for OVSDB Southbound

Patch 2: Change the file contents so that everything is not in a single line.

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

Implement NodeCacheListener to SouthboundHandler

- implemented the NodeCacheLister for SouthboundHandler
- changed FlowCapableNodeDataChangeListener to use the NodeCacheManager
- modified Activator to include the interface
- moved the low frequency logs to info level, moved high-frequency to trace

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

Add class name to log for better debugging

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

Fix toString for different events to avoid NPE

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

Merge "Patch to bring us into alignment with skip.karaf.featureTest from odlparent"

Add controller to bridge

You can now write controller info to a bridge and
read it back in the operational store when it comes
back from ovsdb.

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

Cleanup adsal remanants from integration tests

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

Remove adsal-all from plugin and nsf from openstack feature

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

Remove last adsal remnants from NeutronL3Adapter

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

Merge "Remove final remnants of sal.core from net-virt-provider"

Merge "Remove sal.core from NodeUtils"

Merge "Remove final bits if adsal from plugin-shell"

Merge "Update port in operational datastore."

Remove final bits if adsal from plugin-shell

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

Update port in operational datastore.

This patch updates ports in operational datastore.
There is also a framework for port delete. Future patches will populate this class.

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

Add ability to read and write protocols for bridge

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

Merge "Add support for datapath"

Merge "Initial write for bridge."

Add support for datapath

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

Initial write for bridge.

This code does an initial write of a bridge.
It also introduces a simplifying framework for
handling writes to ovsdb in the southbound plugin.

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

Remove final remnants of sal.core from net-virt-provider

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

Remove sal.core from NodeUtils

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

Merge "Fix to improve logging of exceptions in TransactionInvokerImpl"

Fix to improve logging of exceptions in TransactionInvokerImpl

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

Remove ad-sal from lbaas - part 2

With this change, lbaas handler will be getting notifyNode() callbacks.

Patch 2 (code review):
  - add missing copyright header
  - make node callback more resilient

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

Remove ad-sal from lbaas - part 1

Removing ad-sal inventory listener and switchmanager from lbaas.
With this change, lbaas handler will not be getting notifyNode() callbacks.
That will be addressed on part 2.

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

BUG: Fix missing creation of initial openflow rules upon node creation

While using the registerDataChangeListener for createFlowCapableNodePath, we began to
see cases where onDataChanged provided no entries in getCreatedData() when it should.
To deal with that, the code will now pay attention to updates in getUpdatedData() and
treat it as create if this is the first time the listener hears about the nodeId.

Patch 2:
  - listen to operational instead of configuration mdsal tree;
  - check for create/update inside change loop
  - remove redundant [removed/added] words in log
Patch 3:
  - fix notifyNodeUpdated log
Patch 4:
  - change log level for notifyNodeUpdated
Patch 5:
  - undo patch 4. Lots of logs [1]

[1]: https://gist.github.com/67daada34dd91aff3190

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

net-vir-provider : Remove depdency on sal.utils package.

Patch 2 : Removed dependency over sal.actions - not used.

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

Merge "Fix typo (Don't know why I wanted indentidentifiers ;)"

Merge "Fixing null pointer error on node delete and executor shutdown."

Fixing null pointer error on node delete and executor shutdown.

Signed-off-by: Wojciech Dec <wdec@cisco.com>
Change-Id: Iebad24357c874bc863ee4c17acb99e2ed29ad9c2

Move NodeUtils.java to ovsdb.utils

Patch 2: implement code review comments.

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

Patch to bring us into alignment with skip.karaf.featureTest from odlparent

Sam Hague was kind enough to point out the use of skip.karaf, which is a
brilliant idea incorporated into the odlparent by this patch:

https://git.opendaylight.org/gerrit/#/c/16493/

This patch simply makes sure that -Dskip.karaf.featureTest does the right
thing in ovsdb.

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

Provide FlowCapableNodeDataChangeListener to openstack.netvirt via NodeCacheManager

With this change, netvirt will have a thread safe cache of all flowCapable nodes.
This is needed for the lbaas handler, so it can efficiently find the nodes it needs
to configure, w/out using ad-sal switch manager.

Patch 2:
- add missing Copyright headers
- add code review comments

Note: NodeUtils has been duplicated and needs to be moved to .utils pkg
This will be taken care in gerrit: https://git.opendaylight.org/gerrit/#/c/16487/

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

Fix typo (Don't know why I wanted indentidentifiers ;)

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

Added convenience method for ops.mutate(TypedTable)

Many other ops methods handled TypedTable, so decided
mutate should too :)

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

Added convenience method to TyperUtils for creating typed row

The existing convenience method would only wrap an existing row,
this method simply wraps around a newly created row.

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

Merge "Renaming ovsdb-managed-node-augmentation to ovsdb-bridge-augmentation."

Renaming ovsdb-managed-node-augmentation to ovsdb-bridge-augmentation.

ovsdb yang model was using ovsdb-managed-node for bridges. This was causing confusion and was renamed to ovsdb-bridge.

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

Remove sal IInventoryListener from Southboundhandler

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

Add option to skip karaf tests for southbound

The karaf test is enabled by default and can only be disabled by explicitly add -Dskip.karaf=true to the mvn build command.

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

Merge changes I00b761de,I6b7440b3,Id4a96515

* changes:
  Add utils.servicehelper to netvirt
  Add utils.servicehelper to northbound
  Add utils.servicehelper to integration tests

Use service dependency injection rather than ServiceHelper.

This removes the need for sal.utils.

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

Add utils.servicehelper to netvirt

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

Add utils.servicehelper to northbound

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

Add utils.servicehelper to integration tests

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

Revert "Revert "Make utils.servicehelper a bundle""

This reverts commit b1ac36f07fadb9ce0651714653b8d0817f882760.

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

Revert "Make utils.servicehelper a bundle"

This reverts commit 933dcb0728022a5d20fc1eafea5ed8e83edf83f8.

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

Add ovsdb-features to karaf distribution

The karaf distribution does not currently have any dependencies defined so it is building before some of the ovsdb modules are built. Adding the features dependency ensures that the other modules will build first.

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

Make utils.servicehelper a bundle

The integration tests had trouble with an inline jar so changed it to a bundle.

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

Initial patch to deprecate sal utils Servicehelper

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

Add root pom for utils modules.

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

Migrate net-virt from adsal Node to mdsal Node

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

Re-order feature dependency, so that neutron-service comes before nsf-service for ovsdb-openstack

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

Migrate from adsal Node to mdsal Node

This is a large commit becuase the sal Node is used all over the code. There is no way to build without doing it all in once shot.

Patch Set 2,3: No changes, just testing openstack integration. Something was broken in controller code causing failures. A rebuild this morning pulled in numerous new artifacts and cleared up the issue.

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

Switch node-id's for ovsdb-managed-nodes

Is now

ovsdb://${ip}:${port}/bridge/${bridgeName}

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

Merge "Make BridgeName strongly typed"

Make BridgeName strongly typed

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

Remove <repositories> and <pluginRepositories> sections

It is recommended that developers and servers configure this locally via
settings.xml.

https://lists.opendaylight.org/pipermail/discuss/2015-January/004482.html

Change-Id: I9e2b2e640bdc788ee355066c5bf40cc8cf813ac6
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Pull Maven surefire plugin for offline builds

Change-Id: If6ed6d4f2c37f75dab52f0c30849727cce206493
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Fix placement of config file"

Fix placement of config file

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

Merge "Bug 2438: No serializer found for class.. BooleanBaseType"

Bug 2438: No serializer found for class.. BooleanBaseType

When using the OVSDB Northbound v3 APIs the below error is returned when GET'ting tables that contain boolean columns.

http://{{controllerHost}}:{{controllerPort}}/ovsdb/nb/v3/node/OVS|192.168.120.31:48161/database/Open_vSwitch/table/Bridge

No serializer found for class org.opendaylight.ovsdb.lib.schema.BaseType$BooleanBaseType and no properties discovered to create BeanSerializer (to avoid exception, disable SerializationFeature.FAIL_ON_EMPTY_BEANS) ) (through reference chain: org.opendaylight.ovsdb.lib.schema.GenericTableSchema["columnSchemas"]->java.util.HashMap["stp_enable"]->org.opendaylight.ovsdb.lib.schema.ColumnSchema["type"]->org.opendaylight.ovsdb.lib.schema.AtomicColumnType["baseType"])

The boolean type does not have a public getter/setter since it doesn't need one but this error comes from that.

Configure the object mapper to safely ignore the error.

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

Merge "OVSDB V3 northbound failed to find node"

Merge "getNode fails when id contains |"

Merge "Ovsdb plugin compatibility layer"

OVSDB V3 northbound failed to find node

The Node.fromString(id) no longer works with the latest adsal deprecation changes.

A new helper method getNode() was previously created as a workaround but the v3 code was not using it. This commit fixes that.

Also added unit test for the v3 northbound.

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

getNode fails when id contains |

The split delimiter was not used correctly. The | is the OR operater so it needed to be escaped.

Also added getNode unit test

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

Ovsdb plugin compatibility layer

As of now this layer just proxy API calls to plugin and relay the notification back to all the listener.
All the modules who want to consume to existing plugin interfaces, should move to the compatibility layer,
because plugin interface will change due to it's migration to md-sal.

*.* Although this layer is just proxy, but classes are moved under new package name to avoid conflict
with plugin code. As of now net-virt code is changed to depend on compatibility layer.
*.* Add new feature ovsdb-compatibility-layer to karaf-feature
*.* All integration tests are passing

Patch 3: Rebased and addressed reviewers comment
Patch 4: Fix single feature test for compatibility layer
Patch 5: Fixed integration tests
Patch 6: Added api back that was deleted accidently. Comments and pom.xml cleanup.

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

Merge "Fix for failed to parse versions like 2.3.1-git3282e51"

Merge "OVSDB SB model for port and interface"

Fix for failed to parse versions like 2.3.1-git3282e51

Existing code was matching using the pattern major.minor.patch and would fail if anythign extra was in the string. This was because the code used an exact match across the whole string. Chaning to only find the major.minor.patch pattern fixes the issue by ingoring the extra characters.

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

Add Data Change Listeners for Managed Nodes and Ports

Instantiate data change listeners in the OVSDB
SoutboundProvider for the OVSDB Managed Node and
Port Termination Point augmentations.

v2 - updated against merged OVSDB Termination Point augmentation

v3 - fixed log message in OvsdbTerminationPoint listener

v4 - remove unused imports

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

OVSDB SB model for port and interface

Initial augmentation of termination-point to add
OVSDB port and interface support.  No doubt additional
fields will need to be added.

Looking for feedback on the approach.

v2 - flatten out the model so both ovsdb port and interface
attributes will be present in the ovsdb termination-point
augmentation.  More attributes to be added.

v3 - use identity types for the interface type

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

Fix critical Sonar Issue: OvsdbConnectionService.java

Fix for the following issue:
- Throwable.printStackTrace(...) should never be called

Ref link on logging:
https://wiki.opendaylight.org/view/Logging_Best_Practices

Change-Id: I283f44deb8822b7f445a7cf48b89bec14d3c7df6
Ref: http://sonar.opendaylight.org/component/index#component=org.opendaylight.ovsdb%3Alibrary%3Asrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fovsdb%2Flib%2Fimpl%2FOvsdbConnectionService.java&currentIssue=3c103a64-c1a4-421a-9f4a-17979810815b
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Minor fix - typo in log.

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

Provide a getter for the dbSchema of a TransactionBuilder

This allows code to know whether or not this is a TransactionBuilder
that is of interest to a piece of code based on the schema it
supports.

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

Add getters for OvsdbConnectionInstance to ConnectionManager

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

Merge "MD-SAL OVSDB SB - handle update to ovsdb node."

Merge "Slight simplification in OvsdbConnectionInstance"

Merge "Minor fix to change log message from OvsdbNodeDataChangeListener to OvsdbManagedNodeDataChangeListener"

Fix for incorrect generation of NodeId which was preventing cleanup of nodes
correctly.

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

Slight simplification in OvsdbConnectionInstance

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

Minor fix to change log message from OvsdbNodeDataChangeListener to OvsdbManagedNodeDataChangeListener

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

Use neutron from new repo

Change-Id: I03757e2cebacb3f819e5e8ea7ab0c545ea4053ae
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Fix TransactionBuilder to take OvsdbClient rather than OvsdbClientImpl

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

MD-SAL OVSDB SB - handle update to ovsdb node.

Patch to handle ovsdb node update.

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

Moved extract* methods to TyperUtils

This was discussed with shague on #opendaylight-ovsdb

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

Change signature of TransactionUtils to returns Maps keyed by UUID.

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

Made TransactionUtils easier to use

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

Introducing AbstractTransactionCommand to make it easier to do commands

I found myself writing a lot of repetative code for TransactionCommands,
and so pulled it up into an AbstractTransactionCommand

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

Added Removal of bridges from operational store

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

Switch to command pattern and using transaction chains

This patch does a few things:

1)  Switch to using command pattern with transaction chains
    to provider robust writing to the operational store
2)  Moved over the existing operational writing to using the command
    pattern
3)  Removed the bridges from the ovsdb node as they need to be
    separate topological nodes.

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

Revert "Removing ovsdb plugin dependency on sal.utils classes"

This reverts commit 9ab94103db38d59905e8706e9848ce992721e0eb.

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