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>

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>

Move to use flow models from openflowplugin

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

Fix for ovsdb parent pom version mismatch

Patch 2: retry

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

Merge "Nit pick: fix spacing in getNode()"

Merge "Fetch managed node (bridge) details and augment to operational data store"

Nit pick: fix spacing in getNode()

Change-Id: Ia17d027c1f011abdf3783f3c59f1caa04cd68a52
Co-Author: Sam Hague <shague@redhat.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

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

Merge "Workaround for Node.fromString"

Removing ovsdb plugin dependency on sal.utils classes

This patch copies all the required classes (Status.java & StatusCode.java) from sal.utils and places them in plugin.api package.

Patch set 3: Add review comments.

Change-Id: I7e5ca9ef3480f4c3dcf604e1874f7e980681c2e8
Also-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fetch managed node (bridge) details and augment to operational data store

Patch 2: Fixed schema.openvswitch module loading issues

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

Workaround for Node.fromString

Added getnode(id) API to plugin ConnectionService interface to get the node requested by the id.

Also found some cases in the northbound that were using the fromString so replaced them with getNode.

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

Migrate ovsdb to use features-test from odlparent

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

Remove IPlugin{in,out}Inventory* interfaces.

This patch removes ovsdb plugin module's dependency on IPluginInInventory* and IPluginOutInventory* interfaces.

Patch has TODO item's that will be implemented and pushed in upcoming patches.

Change-Id: I69b9bab711d9e346629ede9ff8eec97989af31cc
Also-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

fix integration tests

patch 2: only re-add pluginRepositories section

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

Merge "Parsing bridge names on new Southbound, Yang changes"

Parsing bridge names on new Southbound, Yang changes

Change-Id: I8a0d1802c37c49b812ce5b5b008d7e22169b3655
Signed-off-by: Reinaldo Penno <rapenno@gmail.com>

Merge "Fix of ovsdb conneciton listener"

Fix of ovsdb conneciton listener

For ssl connection, the notification of connected event to listeners should happen after ssl handshake is complete.

Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>

Clean up pom files.

- Add missing Copyright headers.
- Remove duplicated properties that are pulled from odlparent

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

Merge "Add karaf-based pax-exam so that net-virt can be tested."

Add karaf-based pax-exam so that net-virt can be tested.

Use the following command to run the NetVirtIT test:

mvn -Dit.test=NetVirtIT verify -Pintegrationtest -Dovsdbserver.ipaddress=192.168.120.31 -Dovsdbserver.port=6640 -Pkarafit

This will start the test and bring up the ovsdb karaf distribution. You should first build the distribution and install it to your local m2 repo. When the odl-ovsdb-openstack feature is fully loaded you can then manual configure the bridge to connect to odl with ovs-vsctl set-manager tcp:192.168.120.1:6640.

By default the integration tests exclude NetVirtIt. This is because the current test cannot get service interfaces to make use of their services. An example is getting the connect interfce from the plugin to initiate and outgoing connection to an ovsdb node. The usual "ServiceHelper.getGlobalInstance" did not work in the karaf pax-exam container so another workaround needs to be found.

The test will wait for 10s for the connection from the ovsdb node. Look for the output: "Try to connect!" before connecting. This message indicates the odl-ovsdb-openstack feature is active and ready to receive connections. The timeout can be increased if needed.

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

Re-enabled southbound

It appears to be working fine.

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

Use odlparent for pax-exam dependencies.

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

Bug 2707: Pipeline flows are not programmed because of failed onNodeUpdated call

Cause: we used the onNodeUdpated() OpendaylightInventoryListener interface to receive updates when openflow nodes were updated. If we thought it was the first time and the openflow node was just added we would then program the pipeline flows. ovsdb checked the FlowCapableNode in the datastore for the isPreset flag. It looks like there might have been some race condition where another writer like InventoryManager was updated the FlowCapablNode such that when ovsdb would do the above check isPresent would return true and ignore the update - and not program the pipeline flows.

Solution: replace the deprecated call with an mdsal flowCapablenode DataChangeListener. This ensures ovsdb will get the node when created.

Also added a couple log.info's to track when the nodes are added to help with future debugging.

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

Remove dependency on sfc-model to help with yangtools issues.

Also removed building southbound since it has issue with parent pom. Not sure if it is related to all the other build issues from the recent yangtools commits.

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

Merge "Fix sonar code coverage for BridgeConfigurationManagerImpl.java"

Fix sonar code coverage for BridgeConfigurationManagerImpl.java

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

Adding adsal todo markers to track needed changes.

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

Add copyright header to files missing one

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

OVSDB MD-SAL SB - remove extra disconnected call

When deleting an active ODL connection to an OVS node
via restconf, the disconnect() call in
OvsdbConnectionManager.java called the disconnected()
method which resulted in two transactions to remove the
operational MD-SAL entry for the node.  Which resulted
in WARNings in the log.

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

Added basic framework for getting requests for OvsdbManagedNodes

Provides basic framework for getting requests for OvsdbManagedNodes so
we can translate them to transacts.

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

Merge topics 'mdsal_ovsd_db', 'mdsal_ovsdb_sb'

* changes:
  Provide some simple getters for getting the client from the ConnectionManager
  Switch to using instanceidentifiers for managed-by and manages references

Merge "MD-SAL OVSDB SB - handle deletion of Ovsdb Nodes"

Merge "Basic wiring for OVSDB updates"

Merge "Fix ovsdb-ovs-full-integration-daily"

Merge "Remove ComponentActivatorAbstractBase from net-virt"

Fix ovsdb-ovs-full-integration-daily

Fix issue introduced in https://git.opendaylight.org/gerrit/#/c/15105/
testGetOpenflowVersion() was not accounting for cases when OVS does not support OF13.

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

Provide some simple getters for getting the client from the ConnectionManager

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

Switch to using instanceidentifiers for managed-by and manages references

This is done because instance-indentifiers are easier to work with :)

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

Fix version used for ../commons/parent/commons

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

MD-SAL OVSDB SB - handle deletion of Ovsdb Nodes

This is a first draft for handling deletion of
Ovsdb nodes that were added via restconf.

This first patch cheats to get things working.  Need
a method in disconnect() to find the Ovsdb client.

v2 - fixed problem lloking up Ovsdb client.
v3 - rebased against Ed's patch - gerrit 15043

Change-Id: Ie2804b50c5205c2d1d2ca44fa17a64db3518b8a2
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Remove ComponentActivatorAbstractBase from net-virt

Change-Id: Ie173040211627cfa2c467247fd164aa0965f539a
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Ref: https://git.opendaylight.org/gerrit/#/c/15034/

Basic wiring for OVSDB updates

This patch wires in OVSDB updates via monitor, and hands
them off to a callback that (to date) logs them.

I've tried it out with some simple cases, and it appears to
be working.

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

Merge "Fix namespaces for dpdk yang files"

Merge "Remove ComponentActivatorAbstractBase from plugin"

Fix namespaces for dpdk yang files

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

Remove ComponentActivatorAbstractBase from plugin

Patch Set 3: remove some unused imports.

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

Merge "Remove openflow10"

Revert "Revert "Adding support for DPDK datapath_type and port_types.""

This reverts commit 96cabf057a4ab117c5bf6f615f00841ca262494c.

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

Merge "Re-enable southbound build."

Re-enable southbound build.

After some investigation, this turns out to be a deeply adverse interaction
between pax-exam 4.4.0 (and 4.0.0-4.4.0) and mirrors in the infra.

Since I believe we need to go to at least pax-exam 4.0.0 to get to karaf
3.0.2 to get to Java 8 (or so I recall from talking to Robert), the mirror
thing in infra has been fixed (nobodies preferred fix).

This patch should now verify, and the build should be restored.

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

Merge "Revert "Adding support for DPDK datapath_type and port_types.""

Remove openflow10

Follow up from https://git.opendaylight.org/gerrit/#/c/14968/
to remove dead code newly made.

Also re-enabled test in NeutronIT: testGetOpenflowVersion()

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

Revert "Adding support for DPDK datapath_type and port_types."

It appears this patch only passed verify because southbound was not building.

This reverts commit feec58a6b62fde434e7972b172f254a996bea6c6.

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

Merge "Remove of10provider"

Remove of10provider

This helps to remove the switchmanager and forwardingrulesmanager adsal dependencies. switchmanager is still needed by net-virt so that will be fully removed in a later commit.

Patch set 2: Reenable the NeutronIt and OvsdbNorthboundIT
             Add fix for previous commit with uriInfo that was breaking the OvsdbNorthboundIT
             Incorporate review comments.

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

Adding support for DPDK datapath_type and port_types.

This patch creates yang models to support datapath and port types.
datapath-port-types.yang is a general purpose datapath and port types model.
dpdk.yang is an extension to add dpdk specific ports and datapath_types.

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

Remove sal.connectionservice

Patch Set 4: Add ConnectionConstants.
Patch Set 5: Add pom files.

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

Fix for OVSDB compilation failure

- Legacy distribution was removed but top level pom still had child module
  which causes a fresh tree to fail compilation.

Change-Id: I0b258da4a1516035954a56c913a8368694e5300f
Signed-off-by: Reinaldo Penno <rapenno@gmail.com>

Merge "REVERT-ME: neuter southbound module."

REVERT-ME: neuter southbound module.

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

Remove legacy distribution

I think (I checked with flaviof on IRC) this is just
making build times longer and is not used.  If so,
this patch should make builds faster.

If its doing something useful, let me know ;)

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

SonarBugFix:
Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>
Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>

SonarBugFix:GenericTableSchema
Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>

Sonar bug Fixes

Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>

SonarBug:Remove unused variables

Conflicts:
northbound/src/main/java/org/opendaylight/ovsdb/northbound/OvsdbNorthboundV3.java

Change-Id: I4bbdcc9517966f91230987b09ae7cd27f6d845c2
Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Add more detail to trace logging in JsonRpcEndpoint"

Merge "BUG-1173: do not mention guava.version"

Add more detail to trace logging in JsonRpcEndpoint

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

Merge "Fix for Sonar bugs Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>"

Merge "Rough musings towards an MD-SAL OVSDB SB"

BUG-1173: do not mention guava.version

Guava dependency is declared from odlparent, no need for version here.

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

Fix for Sonar bugs
Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>

Rough musings towards an MD-SAL OVSDB SB

This patch is incomplete, it only manages to accept
ovsdb connections and log them, but its a start.

PatchSet2: Added a simple starting model around tunneling
PatchSet3: Added additional tunnel info
PatchSet4: Added ovsdb.yang model.  Separated tunnel into tunnel.yang
PatchSet5: Renamed tunnel.yang to overlay.yang.  Added topology-type info.
PatchSet6: Added in dummy DataChangeListener for OvsdbNodes.
           Moved mapping functions into a mapping class.
           Handled tracking of OvsdbClients
PatchSet7:  Responded to comments:
            Fixed whitespace
            Fixed parent for aggregator
            Fixed features from odl-southbound- to odl-ovsdb-southbound-
PatchSet10: Added DataChangeListner to listen for new ovsdb-nodes in topology
            and connect to them.
PatchSet15: Response to Colin's comments
PatchSet18: Fixed feature dependencies.  Depends on patch
            https://git.opendaylight.org/gerrit/#/c/14791/ in controller
PatchSet19: Fixed small error in startup order

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

Remove plugin projects dependency on ad-sal clustering.
Currently plugin code is not using any feature of clustering in the existing code. Removing this unnecessary dependency

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

Merge "Fix 'Row cannot be cast to Interface' exceptions"

Fix 'Row cannot be cast to Interface' exceptions

This patch modifies NetworkHandler and PortHandler to call
ovsdbConfigurationService.getTypedRow()

Change-Id: Ifef438257091c56db05508fcad3b8973fd93a7e5
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>

Merge "Remove deprecated comment."

Merge "Added unit test BridgeConfigurationManagerImpl.java"

Remove deprecated comment.

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

Added unit test BridgeConfigurationManagerImpl.java

Patch Set 2: Added dependency for tests to pom.xml
Patch Set 3: Add review comments.

Change-Id: I8bc96fc2dee23e8d7f7b28cb033d6a25ca417297
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Added unit test LoadBalancerConfiguration.java

Patch Set 2: Rebase
Patch Set 3: Add review comments.

Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>
Change-Id: I4d743389eacd7b3d7c2f4f6174ceab90e760c75e

Bug-2658 - Remove adsal and migrate to mdsal

Clean up pom file dependencies.

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

BUg-2658: Remove adsal and migrate to mdsal

Remove networkconfiguration.

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

BUG-2658: Remove adsal and migrate to mdsal

The patch removes the deprecated BridgeDomain api.

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

BUG-2658: Remove adsal and migrate to mdsal

This patch removes the deprecated connection and configuration services.

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

BUG-2658: Remove adsal and migrate to mdsal

This commit covers removing adsal-compatibility.

patchset 2: Remove adsal-compatibility from ovs-sfc features
patchset 3: Really remove the adsal-compatibility from ovs-sfc features.
patchset 4: Apparently not
patchset 5: Trying again

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