netvirt.git
8 years agoMerge "Bug 4996 - Wrong flows when using SFC coexistence"
Sam Hague [Tue, 19 Jan 2016 00:40:04 +0000 (00:40 +0000)]
Merge "Bug 4996 - Wrong flows when using SFC coexistence"

8 years agoBug 4997 - Fix for ConnectionInfo parsing in showMdsal tool
Sam Hague [Mon, 18 Jan 2016 21:59:59 +0000 (16:59 -0500)]
Bug 4997 - Fix for ConnectionInfo parsing in showMdsal tool

Change-Id: If3fee23373829c520fcc532ba70b1a45061aa2b3
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoBug 4996 - Wrong flows when using SFC coexistence
Sam Hague [Mon, 18 Jan 2016 21:58:42 +0000 (16:58 -0500)]
Bug 4996 - Wrong flows when using SFC coexistence

Change-Id: Icb9cab27b271f0e5462ecb87ead7a10f0f083bd1
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoMerge "BUG-4205 : VM delete doesnot remove all related flows"
Flavio Fernandes [Fri, 15 Jan 2016 14:58:48 +0000 (14:58 +0000)]
Merge "BUG-4205 : VM delete doesnot remove all related flows"

8 years agoMerge "Fixes an issue in induvidual security rule update, when remote SG is selected."
Sam Hague [Fri, 15 Jan 2016 14:46:52 +0000 (14:46 +0000)]
Merge "Fixes an issue in induvidual security rule update, when remote SG is selected."

8 years agoBUG-4205 : VM delete doesnot remove all related flows
bviswa [Thu, 22 Oct 2015 03:08:01 +0000 (08:38 +0530)]
BUG-4205 : VM delete doesnot remove all related flows

Patch set 3: Rebase and add ref to bug 4971
Patch set 5: Rebase again and move processSecurityGroupUpdate()

Change-Id: Ia3cbba3087232cb56eee6e5424cdd011a01ce266
Signed-off-by: bviswa <badrinath_viswanatha@dell.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoBump versions by 0.1.0 for next dev cycle
Thanh Ha [Fri, 15 Jan 2016 06:33:11 +0000 (01:33 -0500)]
Bump versions by 0.1.0 for next dev cycle

Change-Id: I647f1e840b4d4bc587f6cbce4018ed187922a7a0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
8 years agoFixes an issue in induvidual security rule update, when remote SG is
Aswin Suryanarayanan [Thu, 14 Jan 2016 19:44:01 +0000 (01:14 +0530)]
Fixes an issue in induvidual security rule update, when remote SG is
selected.

Change-Id: Id8d5ea6c6235c31f9147556244f619a15a279382
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
8 years agoMerge "Add support for bfd-status for monitoring"
Sam Hague [Thu, 14 Jan 2016 11:10:30 +0000 (11:10 +0000)]
Merge "Add support for bfd-status for monitoring"

8 years agoMerge "Workaround for failing Netvirt SG IT"
Sam Hague [Thu, 14 Jan 2016 03:19:22 +0000 (03:19 +0000)]
Merge "Workaround for failing Netvirt SG IT"

8 years agoWorkaround for failing Netvirt SG IT
Sam Hague [Thu, 14 Jan 2016 02:01:23 +0000 (21:01 -0500)]
Workaround for failing Netvirt SG IT

Change-Id: I2e9ed58680ebefb6ceed912af5518b5eb543da7d
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoAdded cluster aware mdsal utils class to net-virt (redo)
Anil Vishnoi [Mon, 11 Jan 2016 18:46:42 +0000 (10:46 -0800)]
Added cluster aware mdsal utils class to net-virt (redo)

Looks like while code cleanup we removed the MdsalUtils.java from the master branch
and while merging master branch to net-virt-clustering, it got removed as well.
Net-virt-clustering had a modified version of MdsalUtils.java, where put/merge/delete
transactions are only done by the master instances of net-virt. ClusterAwareMdsalUtils
class bring those changes back.

Patch set 2: Explicitly callback into OvsdbInventoryService when netVirt becomes owner.
Patch set 3: Undo patch set 2; make ProviderEntityListener to happen earlier

Change-Id: I42ab5f2969bcff7737d90f2651d2762897e74c9a
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoMerge "Revert "Added cluster aware mdsal utils class to net-virt.""
Flavio Fernandes [Thu, 14 Jan 2016 01:11:29 +0000 (01:11 +0000)]
Merge "Revert "Added cluster aware mdsal utils class to net-virt.""

8 years agoRevert "Added cluster aware mdsal utils class to net-virt."
Flavio Fernandes [Wed, 13 Jan 2016 21:34:43 +0000 (21:34 +0000)]
Revert "Added cluster aware mdsal utils class to net-virt."

This reverts commit e9ec56b353a2d9098e77d623b7c756069bb58cb9.

This causes a regression in that netvirt:1 is never written to mdsal. to be re-visited.

Change-Id: I683dcac86c23445762d460183fa3e651a600f2c2
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoMerge "Drop testRegisterNewInterface() entirely"
Flavio Fernandes [Wed, 13 Jan 2016 12:24:12 +0000 (12:24 +0000)]
Merge "Drop testRegisterNewInterface() entirely"

8 years agoAdd support for bfd-status for monitoring
Vishal Thapar [Wed, 13 Jan 2016 11:40:29 +0000 (12:40 +0100)]
Add support for bfd-status for monitoring

bfd-status gives information about current state of tunnels. This required
to monitor tunnels and know when they go up/down.

Change-Id: I5d864d46adf1444b00382acef58e298df5dc155e
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoDrop testRegisterNewInterface() entirely
Stephen Kitt [Wed, 13 Jan 2016 09:55:54 +0000 (10:55 +0100)]
Drop testRegisterNewInterface() entirely

This test requires a bundle context to work properly, so it's not
really doable in a unit test.

Change-Id: I29a5a74597146dc3342c6fe7906a28fe89d6d22a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
8 years agoBug 4132 config ovs to respond to pings to router
Josh [Sun, 10 Jan 2016 11:13:01 +0000 (13:13 +0200)]
Bug 4132 config ovs to respond to pings to router

Relies on https://git.opendaylight.org/gerrit/#/c/32135/
N.B. I added a unit test, IcmpEchoResponderServiceTest,
however it's currently skipped along with all the other
unit tests under services.

Patch set 8: Fix imports

Change-Id: If5d212c29acaf49d3cdd16cb9a02a63073202a88
Signed-off-by: Josh <jhershbe@redhat.com>
8 years agoMerge "Use ClusteredDataTreeListener in hwvtepsb"
Sam Hague [Wed, 13 Jan 2016 00:23:23 +0000 (00:23 +0000)]
Merge "Use ClusteredDataTreeListener in hwvtepsb"

8 years agoMerge "Adding changes to use and support connecting tracking. By default SG will...
Sam Hague [Tue, 12 Jan 2016 23:04:58 +0000 (23:04 +0000)]
Merge "Adding changes to use and support connecting tracking. By default SG will push flows with out conntrack match and action, but user can modify default-config.xml to enable conntrack."

8 years agoDisable broken unit test: NeutronFloatingIPInterfaceTest
Sam Hague [Tue, 12 Jan 2016 21:46:33 +0000 (16:46 -0500)]
Disable broken unit test: NeutronFloatingIPInterfaceTest

The test fails with:

Running org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest
Tests run: 9, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.908 sec <<< FAILURE! - in org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest
testRegisterNewInterface(org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest)  Time elapsed: 0.054 sec  <<< ERROR!
java.lang.NullPointerException: null
        at org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterface.<init>(NeutronFloatingIPInterface.java:35)
                at org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest.testRegisterNewInterface(NeutronFloatingIPInterfaceTest.java:329)

There are other exceptions listed earlier which look to be a conflict between the mockito and jacoco:

https://gist.github.com/02f3a56db00ce9b15eb2

Change-Id: I519e63c4a8825d49189dbc4b241998b940c6f4f2
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoMerge "Enable PowerMock instrumentation with JaCoCo"
Sam Hague [Tue, 12 Jan 2016 20:06:22 +0000 (20:06 +0000)]
Merge "Enable PowerMock instrumentation with JaCoCo"

8 years agoMerge "Added cluster aware mdsal utils class to net-virt."
Sam Hague [Tue, 12 Jan 2016 20:05:54 +0000 (20:05 +0000)]
Merge "Added cluster aware mdsal utils class to net-virt."

8 years agoMerge "Explictly cleanup all the bridges managed by the node"
Sam Hague [Tue, 12 Jan 2016 20:05:28 +0000 (20:05 +0000)]
Merge "Explictly cleanup all the bridges managed by the node"

8 years agoMerge "Unit test file for NeutronFloatingIPInterface."
Sam Hague [Tue, 12 Jan 2016 19:53:29 +0000 (19:53 +0000)]
Merge "Unit test file for NeutronFloatingIPInterface."

8 years agoMerge "Bug 4927 - Rules added by MacResolverServices in br-ex are not removed"
Sam Hague [Tue, 12 Jan 2016 19:52:53 +0000 (19:52 +0000)]
Merge "Bug 4927 - Rules added by MacResolverServices in br-ex are not removed"

8 years agoAdding changes to use and support connecting tracking. By default SG
raveek [Thu, 7 Jan 2016 05:03:00 +0000 (10:33 +0530)]
Adding changes to use and support connecting tracking. By default SG
will push flows with out conntrack match and action, but user can modify
default-config.xml to enable conntrack.

Patch 2: after fixing the review comments
Patch 3: Fixed the test failure

Change-Id: I71d8fd23d66caad97e406507889ebba38bd80d27
Signed-off-by: raveek <raveek@hp.com>
8 years agoMerge " Added new feature *odl-ovsdb-openstack-clusteraware*."
Sam Hague [Tue, 12 Jan 2016 19:50:08 +0000 (19:50 +0000)]
Merge " Added new feature *odl-ovsdb-openstack-clusteraware*."

8 years agoMerge "Add support for table offset"
Sam Hague [Tue, 12 Jan 2016 19:49:21 +0000 (19:49 +0000)]
Merge "Add support for table offset"

8 years agoUse ClusteredDataTreeListener in hwvtepsb
Vishal Thapar [Tue, 12 Jan 2016 19:30:53 +0000 (20:30 +0100)]
Use ClusteredDataTreeListener in hwvtepsb

Change-Id: I41234fd2ece1c618107041888d75dc9cc7a62e16
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoFix npe from getLocalIp
Sam Hague [Tue, 12 Jan 2016 16:35:41 +0000 (11:35 -0500)]
Fix npe from getLocalIp

Change-Id: I68336c2aa284bebb4a09d2fa596de91a1ee2d272
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoAdd support for table offset
Sam Hague [Mon, 11 Jan 2016 19:57:15 +0000 (14:57 -0500)]
Add support for table offset

This commit adds a new config parameter table-offset to
allow setting the openflow table base. The default value is 0.
The user can change the value via the config file
netvirt-providers-impl-default-config.xml or by using the restconf
api show below. The value used must be between 1 and 145 - with 145
coming from 255-110 - 110 is the max table in the netvirt pipeline.

The following curl commands can be used to configure the value:
- Get the current value:
curl -u admin:admin http://localhost:8181/restconf/config/netvirt-providers-config:netvirt-providers-config
- Change the value: In this example the value is changed to 1
curl -u admin:admin http://localhost:8181/restconf/config/netvirt-providers-config:netvirt-providers-config -i -H "Content-Type: application/json" --data '{"netvirt-providers-config": {"table-offset": 1}}' -X PUT

If the table-offset is set, then an extra flow is added to the br-ex and br-int
switches to steer packets from table 0 to the first table in the pipeline.

IT tests were all updated to check for the new table-offsets as well as
adding unti-tests. Some IT tests have a little more due to the Yang
pacth that enforces strictre rules on the Yang files and was failing the tests.

Further patches should add config service in netvirt to be a central point
handling all the config updates. SG is also adding a config value that should
be included.

Change-Id: I44b3a442e8050d8f1ef1c5cd9db5c32af2d5ce19
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoEnable PowerMock instrumentation with JaCoCo
Stephen Kitt [Tue, 12 Jan 2016 15:20:19 +0000 (16:20 +0100)]
Enable PowerMock instrumentation with JaCoCo

By default PowerMock is incompatible with JaCoCo, even with the older
version of JaCoCo defined by odlparent: the tests run but their
coverage isn't measured. This patch forces offline instrumentation
which allows both PowerMock and JaCoCo to operate (more or less)
properly. There are a number of "class already instrumented" errors
but they don't affect the test results.

This patch also upgrades PowerMock to 1.6.4 (initially in the hope
that it would work with JaCoCo as-is); this requires an upgrade to
Mockito too. I'll drive these changes properly in odlparent and remove
the local overrides when appropriate!

Change-Id: I2ab704f7a28f325ed3bb5a75a6f34ffb6c3b36b3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
8 years agoAdded cluster aware mdsal utils class to net-virt.
Anil Vishnoi [Mon, 11 Jan 2016 18:46:42 +0000 (10:46 -0800)]
Added cluster aware mdsal utils class to net-virt.

Looks like while code cleanup we removed the MdsalUtils.java from the master branch
and while merging master branch to net-virt-clustering, it got removed as well.
Net-virt-clustering had a modified version of MdsalUtils.java, where put/merge/delete
transactions are only done by the master instances of net-virt. ClusterAwareMdsalUtils
class bring those changes back.

Change-Id: I1d5353572c408e519d7a3cc75356b73436dae147
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
8 years agoExplictly cleanup all the bridges managed by the node
Anil Vishnoi [Mon, 11 Jan 2016 18:01:40 +0000 (10:01 -0800)]
Explictly cleanup all the bridges managed by the node

Change-Id: I37d608a7bb1096d7291d12a0d50dd482f257542f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
8 years agoBug 4927 - Rules added by MacResolverServices in br-ex are not removed
Flavio Fernandes [Tue, 12 Jan 2016 01:58:58 +0000 (20:58 -0500)]
Bug 4927 - Rules added by MacResolverServices in br-ex are not removed

This bug can be reproduced by using an external router address
that is not owned by any system in the external neutron subnet.

Made the flow remove call idem-potent and in a common function.

Added missing flow remove calls in the following 2 missing places:
  * Entry removal (non periodic and periodic)
  * Time period restart

Change-Id: Idf481d3a13c1728bf400f185de296cbada7d920b
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit b1cf5de78228b23a4c6aae1613df925acfa798c9)

8 years agoBug 4331 - External bridge used for gateway mac resolver needs to be dynamic
Flavio Fernandes [Fri, 8 Jan 2016 03:57:33 +0000 (22:57 -0500)]
Bug 4331 - External bridge used for gateway mac resolver needs to be dynamic

With this change the GatewayMacResolverService will become resilient to the cases
where nodes get deleted or lose br-ex connectivity after the resolver is kicked
off to periodically track a given ip address.

Also made some minor clean up in NeutronL3Adapter, so that it needs not to
duplicate work for tracking ip in the very first iteration. That duplication
is not necessary as the GatewayMacResolverService is already implemented
with thread executors for the periodic monitoring.

Lastly, this gerrit augments the GatewayMacResolver service to provide
an extra boolean param called refreshExternalNetworkBridgeDpidIfNeeded.
That can be used to control whether dpid used gets updated when mac is
not getting resolved.

Change-Id: I95b9e5a2d5773a6be82221b71000ace2a662e772
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoMerge "Bug 4913 - populate openflow rules for security group on port creation"
Sam Hague [Sun, 10 Jan 2016 21:52:50 +0000 (21:52 +0000)]
Merge "Bug 4913 - populate openflow rules for security group on port creation"

8 years agoMerge "Bug 4912 - NeutronL3Adapter: missing/reordered enabled check"
Sam Hague [Sun, 10 Jan 2016 21:50:22 +0000 (21:50 +0000)]
Merge "Bug 4912 - NeutronL3Adapter: missing/reordered enabled check"

8 years agoMerge "Fix UT: call method under test and verify"
Sam Hague [Sun, 10 Jan 2016 15:28:06 +0000 (15:28 +0000)]
Merge "Fix UT: call method under test and verify"

8 years agoMerge "Bug 4881 - failure in netvirt.AbstractEventTest"
Sam Hague [Sun, 10 Jan 2016 13:08:12 +0000 (13:08 +0000)]
Merge "Bug 4881 - failure in netvirt.AbstractEventTest"

8 years agoMerge "Bug 4911 - unbreak a55db97e8ce43aec9e2f3a3fe70f6bec3272195b"
Sam Hague [Sun, 10 Jan 2016 13:06:26 +0000 (13:06 +0000)]
Merge "Bug 4911 - unbreak a55db97e8ce43aec9e2f3a3fe70f6bec3272195b"

8 years agoMerge "Add netvirt and hwgw features to main ovsdb/features."
Sam Hague [Sun, 10 Jan 2016 13:00:43 +0000 (13:00 +0000)]
Merge "Add netvirt and hwgw features to main ovsdb/features."

8 years agoMerge "QoS Schema - fix type of QoS type"
Sam Hague [Sun, 10 Jan 2016 12:59:59 +0000 (12:59 +0000)]
Merge "QoS Schema - fix type of QoS type"

8 years agoMerge "Remove unused SchemaVersionMismatchException constructors"
Sam Hague [Sun, 10 Jan 2016 12:59:51 +0000 (12:59 +0000)]
Merge "Remove unused SchemaVersionMismatchException constructors"

8 years agoMerge "Bug 4874 - distributed arp in old l3 for ovsdb is not installing rules when...
Sam Hague [Sun, 10 Jan 2016 12:59:09 +0000 (12:59 +0000)]
Merge "Bug 4874 - distributed arp in old l3 for ovsdb is not installing rules when it should"

8 years agoMerge "Add a unit test for TyperUtils::checkVersion()"
Sam Hague [Sun, 10 Jan 2016 12:58:04 +0000 (12:58 +0000)]
Merge "Add a unit test for TyperUtils::checkVersion()"

8 years agoRemove failing SouthboundIT test because of new yang enforcement
Sam Hague [Sun, 10 Jan 2016 03:37:46 +0000 (22:37 -0500)]
Remove failing SouthboundIT test because of new yang enforcement

https://bugs.opendaylight.org/show_bug.cgi?id=4355 added enforcement of values
marked as mandatory in a leaf. The failing tests were negative testing for
setting external_ids with an empty value. That test is invalid now that the
mandatory value is enforced.

Missed a instance in this patch: https://git.opendaylight.org/gerrit/#/c/32321/
but it was already merged.

Change-Id: I5fcc140907b4deec69e025a5055963f4a6743a01
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoMerge "Disable hwvtep tests until impl is ready"
Sam Hague [Sun, 10 Jan 2016 03:40:25 +0000 (03:40 +0000)]
Merge "Disable hwvtep tests until impl is ready"

8 years agoRemove failing SouthboundIT test because of new yang enforcement
Sam Hague [Sun, 10 Jan 2016 01:56:54 +0000 (20:56 -0500)]
Remove failing SouthboundIT test because of new yang enforcement

https://bugs.opendaylight.org/show_bug.cgi?id=4355 added enforcement of values
marked as mandatory in a leaf. The failing tests were negative testing for
setting external_ids with an empty value. That test is invalid now that the
mandatory value is enforced.

Change-Id: If6cdcd8a899a6fcde5ca00a2ad2c505e728499e9
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoMerge "Disable hwvtepsouthbound-it"
Sam Hague [Sat, 9 Jan 2016 15:42:25 +0000 (15:42 +0000)]
Merge "Disable hwvtepsouthbound-it"

8 years agoBug 4913 - populate openflow rules for security group on port creation
Isaku Yamahata [Sat, 9 Jan 2016 01:45:53 +0000 (17:45 -0800)]
Bug 4913 - populate openflow rules for security group on port creation

When port is created, original port is null.
In that case, flow should be populated instead of ignoring the events.

Change-Id: I1f90868bbb4bcb3e7d40474133bcaafe2fc1983a
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
8 years agoBug 4912 - NeutronL3Adapter: missing/reordered enabled check
Isaku Yamahata [Sat, 9 Jan 2016 02:05:40 +0000 (18:05 -0800)]
Bug 4912 - NeutronL3Adapter: missing/reordered enabled check

NeutronL3Adapter.processEvent() can be called before its initialization
The security group process should be after enabled check.

Change-Id: I7fc48a70cacca5a9a1d7a017476259543c8a0086
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
8 years agoBug 4911 - unbreak a55db97e8ce43aec9e2f3a3fe70f6bec3272195b
Isaku Yamahata [Sat, 9 Jan 2016 02:15:49 +0000 (18:15 -0800)]
Bug 4911 - unbreak a55db97e8ce43aec9e2f3a3fe70f6bec3272195b

The change set of a55db97e8ce43aec9e2f3a3fe70f6bec3272195b,
change id of Ib740544ddbbb08745aa7f21aa9246acad1ad1d53
breaks ipv6 work around as follows.
This patch unbreaks it and apply same change to similar work around.

> java.lang.IllegalArgumentException: Could not parse [2003:0:0:0:0:0:0:1/64]
>         at org.apache.commons.net.util.SubnetUtils.calculate(SubnetUtils.java:240)[242:org.apache.commons.net:3.3.0]
>         at org.apache.commons.net.util.SubnetUtils.<init>(SubnetUtils.java:52)[242:org.apache.commons.net:3.3.0]
>         at org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.services.RoutingService.programRouterInterface(RoutingService.java:62)[279:org.opendaylight.ovsdb.openstack.net-virt-providers:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programRouterInterfaceStage2(NeutronL3Adapter.java:1195)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programRouterInterfaceStage1(NeutronL3Adapter.java:1183)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programFlowForNetworkFromExternal(NeutronL3Adapter.java:1080)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programFlowsForNeutronRouterInterface(NeutronL3Adapter.java:1054)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleNeutronRouterInterfaceEvent(NeutronL3Adapter.java:522)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleNeutronPortEvent(NeutronL3Adapter.java:483)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleInterfaceEvent(NeutronL3Adapter.java:796)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.handleInterfaceUpdate(SouthboundHandler.java:91)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processPortUpdate(SouthboundHandler.java:299)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processPortEvent(SouthboundHandler.java:287)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processEvent(SouthboundHandler.java:232)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.dispatchEvent(EventDispatcherImpl.java:95)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.access$200(EventDispatcherImpl.java:27)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl$1.run(EventDispatcherImpl.java:56)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
>         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)[:1.8.0_66]
>         at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0_66]
>         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)[:1.8.0_66]
>         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)[:1.8.0_66]
>         at java.lang.Thread.run(Thread.java:745)[:1.8.0_66]

Change-Id: I979f7538c9eb8686e9de2773e70514dfb90a4892
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
8 years ago Added new feature *odl-ovsdb-openstack-clusteraware*.
Anil Vishnoi [Sat, 9 Jan 2016 01:47:51 +0000 (17:47 -0800)]
 Added new feature *odl-ovsdb-openstack-clusteraware*.

This feature loads all the required component to make
net-virt cluster aware, like openflowplugin-*-li
features that support clustering
NOTE: THIS FEATURE DISABLE THE AUTO LOAD OF
odl-ovsdb-openstack AT BOOT TIME. USER NEED
TO EXPLICITLY INSTALL THE REQUIRED FEATURE.

Change-Id: Ie087ad3d8b805838125e25d42d87e1ffb1758414
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
8 years agoMerge 'topic/master/net-virt-networking' to 'master'
Anil Vishnoi [Sat, 9 Jan 2016 00:15:19 +0000 (16:15 -0800)]
Merge 'topic/master/net-virt-networking' to 'master'

Squashed commit of the following:

commit 11abd792240e8df1a51a012ba1b92dd7c920c38b
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Fri Jan 8 16:11:50 2016 -0800

    Revert to old openflowplugin

    Change-Id: Icaa1fa04fcc319b128e6f1ee0d58b940385812f3
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 83e3dab5337d20168280ba6018ed0b811c713314
Merge: 6801901 9484b0d
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Mon Jan 4 17:14:55 2016 -0800

    Merge branch 'master' into topic/master/net-virt-clustering

    Change-Id: I4aa7a2d4d10df28f15edcedf5e7a74f16a439ab5
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 68019016506b78adef37f7c7764c0622f5bfb61e
Merge: 23295db 9b3005b
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Sat Jan 2 13:19:59 2016 -0800

    Merge branch 'master' into topic/master/net-virt-clustering

    Change-Id: I29e864f4c09862dc8e41110a77479e6ee6d0632a
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 23295dbb6f26fc0b24521949a14743ddad6d45fb
Merge: a1c8e2b eced0bd
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Dec 29 15:52:23 2015 -0800

    Merge branch 'master' into topic/master/net-virt-clustering

    Change-Id: Ib465c8f20f200d407caaa9440bbf635d72776fe1
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit a1c8e2b2a9738ae946731fb4d111f152057d3b71
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Tue Dec 22 15:39:00 2015 -0500

    Remove extra return statement.

    When Flow id caches that were no longer needed
    were removed from code, a lonely return statement
    in the router interface remove code path was
    left behind.

    Gerrit: https://git.opendaylight.org/gerrit/#/c/29204/

    Change-Id: Ic978953ab65da91baf543e21ea86333b4a9ee5ca
    Reference: Ic2acced7a9a006e6e04afba6fea22aaf0620f134
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit 00de61c7b19937c5b48c1af4d50ae73cdb92b809
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date:   Tue Dec 15 21:23:49 2015 -0800

    Bug 4789 - allowed address pair doesn't have port id

    So getPortId, setPortId methods aren't defined.
    This is the ovsdb part of Bug 4789.

    Change-Id: Id9ad4762b5ed92c5b84e6fab1d7c5135782ca528
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
    (cherry picked from commit 2065f152079fb96f494518efc2b46f4c77db181d)

commit 08ba6afc0ae524716656e5e926bf34675a2e0960
Merge: 4b909d1 9bbb7db
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Mon Dec 14 13:53:12 2015 -0800

    Merge branch 'master' into topic/master/net-virt-clustering

    Conflicts:
     features/ovsdb/pom.xml
     features/src/main/features/features.xml
     openstack/net-virt-providers/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/AbstractServiceInstance.java
     openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtils.java
     openstack/net-virt/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtilsTest.java

    Change-Id: I68702767c0fcae8cb9721805e524d3ecb7889ed4
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 4b909d144f06d4924792c42f918e1f0ce1a86137
Merge: ddbb2eb 5483b28
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Fri Nov 20 00:34:30 2015 +0530

    Merge branch 'master' into topic/master/net-virt-clustering

    Conflicts:
     features/ovsdb/src/main/features/features.xml

    Change-Id: Ib614363bab8752913d430a86b420ab104010243c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit ddbb2eb2a22f271bf57e7470ef1420bd2301193d
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Thu Nov 19 04:50:03 2015 +0530

    Switch to Alternate design of openflowplugin,
    because alternate design supports clustering

    Change-Id: Ia4e14bef316b2576298f013032232dc1e5d64984
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 13264eda1627d01fc6090008e326bcb91bb5e8a8
Merge: 99b5553 c66b0ff
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Nov 17 01:05:38 2015 +0530

    Merge branch 'master' into topic/master/net-virt-clustering

    Conflicts:
     openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/EgressAclServiceTest.java
     openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtils.java
     openstack/net-virt/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtilsTest.java

    Change-Id: I587fc39e1af228c8cff968ab2e5004285ae5bae2
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 99b5553d84f642d80890546643543a65a262645a
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Wed Oct 28 01:52:33 2015 +0530

    Net-Virt Clustering : Enabled remote notifications
    and modify the data store write methods to write only if
    the instance is master instance

    Change-Id: Icee31e0ea829f40bd64a072d840d03f8749c803f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 1e4144b81f47fad768141e7504ce7c2fb1e5f13f
Merge: 218e79b dc9f403
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Tue Oct 27 03:56:39 2015 +0530

    Merge branch 'master' into topic/master/net-virt-clustering

    Change-Id: I1cb9813a044d16dcfa152f6322271bcafe6bfd5c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit 218e79b7bcf213ce64d2b55ff19e208015798aaf
Merge: f91f1a3 6577da8
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Mon Oct 26 22:02:45 2015 +0530

    Merge branch 'master' into topic/master/net-virt-clustering

    Conflicts:
     openstack/net-virt-providers/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/providers/NetvirtProvidersProvider.java

    Change-Id: Ibc577f41a59c24ddc2b4cdb94b4a393f961e0950
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit f91f1a3883bf5a0d8a004e8d5141d2bdffd4aaa3
Merge: fa8fe2a 342c8be
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Sat Oct 17 02:18:56 2015 +0530

    Merge branch 'master' into topic/master/net-virt-clustering

    Conflicts:
     openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/EgressAclServiceTest.java
     openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/IngressAclServiceTest.java
     openstack/net-virt-sfc/features/pom.xml
     openstack/net-virt-sfc/features/production/src/main/features/features.xml
     openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/NetvirtSfcProvider.java
     southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/AbstractTransactionCommandTest.java
     southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OpenVSwitchUpdateCommandTest.java
     southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbBridgeRemovedCommandTest.java
     southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbBridgeUpdateCommandTest.java
     southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbControllerRemovedCommandTest.java

    Change-Id: I3cbe7ef4b0064bf07a489f2561630c4f669695fe
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit fa8fe2a789c741b2026e2946f08bf1e489db4794
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Thu Oct 8 16:17:01 2015 -0400

    Wire EntityOwnership service into OVSDB NetVirt (part 2)

    Change-Id: I222177c38ab8efeccd63924842d166633e88f606
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
    (cherry picked from commit 5239ecfcf1279e72d6d2446d4c88522461b28b1a)

commit a93e284c51d476a1b3b8d4dd9ec532e0caa1c36f
Author: Flavio Fernandes <ffernand@redhat.com>
Date:   Thu Oct 8 13:35:40 2015 -0400

    Wire EntityOwnership service into OVSDB NetVirt

    Implemented logic for netvirt provider to participate in entityOwnership
    service.

    Change-Id: I8f014591099a952d49584eede07624f37492c27d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
    (cherry picked from commit e43de9f997323cfe804857a0f68a01d95641fbc5)

commit e74ec764cc75b295dc3e5babd36f44bb5d930b04
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date:   Thu Oct 8 20:03:36 2015 +0530

    Merge 'master' branch to 'topic/master/net-virt-clustering'
    Squashed commit of the following:

    commit 0cbdadc0af848566e34e66ae7a7ac952fd935561
    Merge: b02c878 cfcc6d4
    Author: Sam Hague <shague@redhat.com>
    Date:   Wed Oct 7 22:42:02 2015 +0000

        Merge "UT for southbound/transactions/md - set 2"

    commit b02c8783853d9308eb704ed7b2ca26d4d6d96933
    Merge: b063584 bf288f9
    Author: Flavio Fernandes <ffernand@redhat.com>
    Date:   Wed Oct 7 21:23:35 2015 +0000

        Merge "If controller joins the cluster late, and the owner of the device is already decided, EntityOwnershipService won't notify new candidate that register for ownership. Adding a proactive check on the ownership state of the device, so if it's already owned, adding the connection instances in the connection cache."

    commit cfcc6d4b4288e44bdbcb9de4cf5ac0de97f741f6
    Author: rpujar <rpujar@inocybe.com>
    Date:   Wed Oct 7 16:29:28 2015 -0400

        UT for southbound/transactions/md - set 2

        4. OvsdbBridgeUpdateCommand.java
        5. OvsdbControllerRemovedCommand.java

        Change-Id: Idc469c08351b1e98129dd9e798c07a660d782709
Signed-off-by: rpujar <rpujar@inocybe.com>
    commit b063584cc53ebf9c072bcb3171ef6d68bb727a8e
    Author: Sam Hague <shague@redhat.com>
    Date:   Wed Oct 7 13:22:54 2015 -0400

        Cleanup netvirtsfc poms

        Change-Id: I6c1a3aa44c75f98d226dcf23dd514a31f5df5d2a
Signed-off-by: Sam Hague <shague@redhat.com>
    commit f5382918beb8096bfb68acf73084a3083c2e51d7
    Author: Sam Hague <shague@redhat.com>
    Date:   Wed Oct 7 13:22:13 2015 -0400

        Bump mdsal-utils version to match other bundles

        Change-Id: I6c5166ae27fd2f055217be95630cbc482261bb61
Signed-off-by: Sam Hague <shague@redhat.com>
    commit 7db67352c22b8275e81c0e919aa661eaa0caa1a3
    Merge: f183e98 ba1ac04
    Author: Sam Hague <shague@redhat.com>
    Date:   Wed Oct 7 13:38:27 2015 +0000

        Merge "UT for southbound/transactions/md"

    commit f183e9857b3a53645b2c463204a4f6945bb10cdb
    Merge: 3a2524e fc26e9e
    Author: Sam Hague <shague@redhat.com>
    Date:   Wed Oct 7 00:36:02 2015 +0000

        Merge "Fix checkstyle issues in netvirtsfc"

    commit bf288f954947f35aca6425f7d180b4732dee6818
    Author: Anil Vishnoi <vishnoianil@gmail.com>
    Date:   Wed Oct 7 02:11:25 2015 +0530

        If controller joins the cluster late, and the owner of the device
        is already decided, EntityOwnershipService won't notify new candidate that register
        for ownership. Adding a proactive check on the ownership state of the device,
        so if it's already owned, adding the connection instances in the connection cache.

        Change-Id: I7e87b55599f0ebd408814d72789c86f231b7907c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
    commit ba1ac0433a93468260f0e00e693ba1a928c14e14
    Author: rpujar <rpujar@inocybe.com>
    Date:   Tue Oct 6 15:56:23 2015 -0400

        UT for southbound/transactions/md

        1. AbstractTransactionCommand.java
        2. OpenVSwitchUpdateCommand.java
        3. OvsdbBridgeRemovedCommand.java

        Change-Id: I7655dec526bc06b770c533dd082eb403bc89a2c7
Signed-off-by: rpujar <rpujar@inocybe.com>
    commit 3a2524e329b4955b96aa59eb8dbb8965d06f08a5
    Merge: 99f9a37 e4783a5
    Author: Sam Hague <shague@redhat.com>
    Date:   Tue Oct 6 19:48:18 2015 +0000

        Merge "Send a canonical IPv4 prefix in table 60 routing rules"

    commit fc26e9e4667e470658ebcc9040f1274903075dcd
    Author: Sam Hague <shague@redhat.com>
    Date:   Tue Oct 6 15:45:13 2015 -0400

        Fix checkstyle issues in netvirtsfc

        Change-Id: I7db518959449958849a74c49a83bde1135ee325f
Signed-off-by: Sam Hague <shague@redhat.com>
    commit e4783a5d2eca5670253b8251d62258edb01eb477
    Author: Eric Multanen <eric.w.multanen@intel.com>
    Date:   Fri Oct 2 06:43:04 2015 -0700

        Send a canonical IPv4 prefix in table 60 routing rules

        Testing with userspace OVS with DPDK uncovered that the routing
        rules that go into table 60 were getting rejected.

        Messages from ovs-vswitchd.log:

        ... |nx_match|WARN|Rejecting NXM/OXM entry
            0:32768:12:1:8 with 1-bits in value for bits wildcarded by the mask.

        ... |connmgr|INFO|br-int<->tcp:10.11.21.7:6653:
            sending OFPBMC_BAD_WILDCARDS error reply to OFPT_FLOW_MOD message

        Clearing the non-prefix bits resolved the problem and the flows were
        successfully added.

        v2 - split up long line into two more readable lines

        Change-Id: I42a0c63e6e837197d29bb8333f12e822a5f15d91
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
    commit 99f9a373e9bc410933421692148b87f177ab4879
    Author: sangeeta.maurya <sangeeta.maurya@hp.com>
    Date:   Tue Oct 6 22:56:55 2015 +0545

        Custom ICMP SG Rule:
        1)Modification in ICMP matcher
        2)Unit Test Improvement (Check added to validate if some values are set
        properly).

        Change-Id: I31fad80f6dd278372abe2b1a6aac95935aef5f17
Signed-off-by: sangeeta.maurya <sangeeta.maurya@hp.com>
    Change-Id: I1b720089469f928ead3a317172901c6ce09ce1ef
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Change-Id: I2b79a3309cf29cf497edb3b8a4b5b5bcdaec9e18
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
8 years agoUnit test file for NeutronFloatingIPInterface.
karthik_j [Mon, 4 Jan 2016 01:46:23 +0000 (07:16 +0530)]
Unit test file for NeutronFloatingIPInterface.

Change-Id: Ief42515748a32c2997bd7c6f43e5ae2b10321d50
Signed-off-by: karthik_j <karthik_j@hcl.com>
8 years agoDisable hwvtep tests until impl is ready
Sam Hague [Thu, 7 Jan 2016 02:04:20 +0000 (21:04 -0500)]
Disable hwvtep tests until impl is ready

Change-Id: I9bf643e55f59413ccb8220fdfc67c1b1b810a092
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoBug 4881 - failure in netvirt.AbstractEventTest
Isaku Yamahata [Wed, 6 Jan 2016 01:12:10 +0000 (17:12 -0800)]
Bug 4881 - failure in netvirt.AbstractEventTest

build fails with the following failure.
transaction id of the event can't be assumed to be 1 because other
tests may have created (subclass of) abstract event before running
this test. This patch to get transaction id from the event and use it
instead of hard-coded 1.

> Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.029 sec <<< FAILURE! - in org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest
> testAbstractEvent(org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest)  Time elapsed: 0.017 sec  <<< FAILURE!
> org.junit.ComparisonFailure: Error, toString() did not return the correct value expected:<...vent [transactionId=[1] handlerType=SOUTHBO...> but was:<...vent [transactionId=[5] handlerType=SOUTHBO...>
>         at org.junit.Assert.assertEquals(Assert.java:115)
>         at org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest.testAbstractEvent(AbstractEventTest.java:59)

Change-Id: I1f351b92a684e8ece17246d8f0011cbffe95409f
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
8 years agoAdd netvirt and hwgw features to main ovsdb/features.
Andre Fredette [Mon, 4 Jan 2016 21:58:44 +0000 (16:58 -0500)]
Add netvirt and hwgw features to main ovsdb/features.

Also:
- Deleted ovsdb/netvirt/{artifacts,features,impl,karaf}.
- Update revision dates on yang files.

Change-Id: I61910ac934f806744bbb5c8d6c75daa8280d61f6
Signed-off-by: Andre Fredette <afredette@redhat.com>
8 years agoDisable hwvtepsouthbound-it
Vishal Thapar [Tue, 5 Jan 2016 15:38:37 +0000 (16:38 +0100)]
Disable hwvtepsouthbound-it

Change-Id: I2a7537721240e8ef98c513771d0f7001f3206f31
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoRemove unused SchemaVersionMismatchException constructors
Stephen Kitt [Tue, 5 Jan 2016 10:00:52 +0000 (11:00 +0100)]
Remove unused SchemaVersionMismatchException constructors

Change-Id: I9ad97e6332b2db75ae080a09aa4d9199ba89adb2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
8 years agoAdd a unit test for TyperUtils::checkVersion()
Stephen Kitt [Tue, 5 Jan 2016 09:59:54 +0000 (10:59 +0100)]
Add a unit test for TyperUtils::checkVersion()

Change-Id: Ibb133dfc0a225c275a9cd6cbdb1fb060a8e97e17
Signed-off-by: Stephen Kitt <skitt@redhat.com>
8 years agoQoS Schema - fix type of QoS type
Eric Multanen [Tue, 15 Dec 2015 22:32:11 +0000 (14:32 -0800)]
QoS Schema - fix type of QoS type

It looks like the 'type' field in the QoS table schema
was incorrectly coded in the Qos.java schema file
to be a Set<String>.
In the ovs schema, the type field for Qos is defined
as a simple string:

        "type": {
         "type": "string"},

Other string fields in the ovs schema that are
defined in the <table>.java files as Set<String>
appear in the schema as follows (for example,
ovs_version from Open_vSwitch table):

       "ovs_version": {
         "type": {"key": {"type": "string"},
                  "min": 0, "max": 1}},

With the QoS 'type' field defined as Set<String>,
the code gets an exception trying to cast
a String to a Set<String> when trying to get
the type out of a table update.  After this patch
is applied, the error is resolved.

Change-Id: Ibb42db03e71d8f1d96329b5942e7501069af20cb
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
8 years agoBug 4874 - distributed arp in old l3 for ovsdb is not installing rules when it should
Flavio Fernandes [Mon, 4 Jan 2016 18:52:05 +0000 (13:52 -0500)]
Bug 4874 - distributed arp in old l3 for ovsdb is not installing rules when it should

Fix TenantNetworkManagerImpl::isTenantNetworkPresentInNode(), so it does not
use stale node to determine the termination points.

Patch set 2: fix UT

Change-Id: I22e1e88ca5b37f6da28b2281f01aaec07c3f751e
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoFix UT: call method under test and verify
Rashmi Pujar [Mon, 4 Jan 2016 18:31:50 +0000 (13:31 -0500)]
Fix UT: call method under test and verify

Change-Id: I7f4dedfb761b23abc04909e5ed6a5a9330c9ee72
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
8 years agoAdded more ITs to hwvtep
Vishal Thapar [Thu, 24 Dec 2015 08:58:59 +0000 (14:28 +0530)]
Added more ITs to hwvtep

Adds following tests and required utils code:
1. testHwvtepNodeConnectDisconnect
2. testAddDeletePhysicalSwitch

Fixes issues found while writing and running ITs

Change-Id: Ia1b4bb03e228dd16c7b01dedc0ef1a4b3d3ea256
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoAdd Tunnel configuration
Vishal Thapar [Sun, 27 Dec 2015 13:43:58 +0000 (19:13 +0530)]
Add Tunnel configuration

Yang changes to configure tunnels [including bfd]
Code to store tunnels information in Operational DataStore.
Code to configure tunnels

Change-Id: Ibd882c6737305af9931488e4cda394e92bcdaf91
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoMerge "Added support for port range in TCP and UDP security group rules"
Sam Hague [Thu, 24 Dec 2015 17:48:15 +0000 (17:48 +0000)]
Merge "Added support for port range in TCP and UDP security group rules"

8 years agoAdded support for port range in TCP and UDP security group rules
Aswin Suryanarayanan [Fri, 25 Dec 2015 00:37:15 +0000 (06:07 +0530)]
Added support for port range in TCP and UDP security group rules

Change-Id: I69ecc8eb52bedb641a7b2c4407b3e070ed2c1677
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
8 years agoFix hwvtep.yang
Vishal Thapar [Thu, 24 Dec 2015 09:53:05 +0000 (15:23 +0530)]
Fix hwvtep.yang

BFD configuration and parameters are defined as string. This is incorrect
and makes it difficult to use BFD cofigs and params. This fix creates
them as lists of key:value pairs of type string:string.

Change-Id: I8af37e443cc04d3665ef4c70824b02d71c14b9b5
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoMerge "First netvirt renderer framework: Hardware Gateway (hwgw)"
Sam Hague [Wed, 23 Dec 2015 23:37:30 +0000 (23:37 +0000)]
Merge "First netvirt renderer framework: Hardware Gateway (hwgw)"

8 years agoMerge "Fix NPE in VLAN Bindings deletion"
Sam Hague [Wed, 23 Dec 2015 23:29:43 +0000 (23:29 +0000)]
Merge "Fix NPE in VLAN Bindings deletion"

8 years agoITs for hwvtepsouthbound
Vishal Thapar [Fri, 18 Dec 2015 06:00:07 +0000 (11:30 +0530)]
ITs for hwvtepsouthbound

Enables IT for hwvtepsouthbound
adds hwvtepsouthboundutils to utils

Change-Id: I365604e75aa45c9a67312b2b275b4041141d75e1
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
8 years agoFix NPE in VLAN Bindings deletion
karanrajsingh [Wed, 23 Dec 2015 10:16:13 +0000 (15:46 +0530)]
Fix NPE in VLAN Bindings deletion

Fixed NPE for multiple physical port addition.

Change-Id: I4b7324bd00dc086d843bb9f1327683dcac696e3d
Signed-off-by: karanrajsingh <karan.raj.s.singh@ericsson.com>
8 years agoFirst netvirt renderer framework: Hardware Gateway (hwgw)
Andre Fredette [Tue, 22 Dec 2015 23:52:30 +0000 (18:52 -0500)]
First netvirt renderer framework: Hardware Gateway (hwgw)

Change-Id: Ic84ba7c7fda140e1bba304541da2d7cc4c0254de
Signed-off-by: Andre Fredette <afredette@redhat.com>
8 years agoMerge "Framework for netvirt model (Part 1)."
Sam Hague [Wed, 23 Dec 2015 02:35:54 +0000 (02:35 +0000)]
Merge "Framework for netvirt model (Part 1)."

8 years agoFramework for netvirt model (Part 1).
Andre Fredette [Mon, 21 Dec 2015 21:53:10 +0000 (16:53 -0500)]
Framework for netvirt model (Part 1).

The structure planned is as follows:

/ovsdb/netvirt
/ovsdb/netvirt/api
/ovsdb/netvirt/artifacts
/ovsdb/netvirt/features
/ovsdb/netvirt/impl
/ovsdb/netvirt/karaf
/ovsdb/netvirt/it
/ovsdb/netvirt/renderers
/ovsdb/netvirt/renderers/hwgw
/ovsdb/netvirt/renderers/neutron
/ovsdb/netvirt/renderers/ovs

This patch contains the "netvirt" directory.

Change-Id: I03ba77273f6193566df63795230db7bd4e20b2e7
Signed-off-by: Andre Fredette <afredette@redhat.com>
8 years agoRemove extra return statement.
Flavio Fernandes [Tue, 22 Dec 2015 20:39:00 +0000 (15:39 -0500)]
Remove extra return statement.

When Flow id caches that were no longer needed
were removed from code, a lonely return statement
in the router interface remove code path was
left behind.

Gerrit: https://git.opendaylight.org/gerrit/#/c/29204/

Change-Id: Ic978953ab65da91baf543e21ea86333b4a9ee5ca
Reference: Ic2acced7a9a006e6e04afba6fea22aaf0620f134
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
8 years agoMerge "Use initFlowBuilder for SfcClassifier"
Sam Hague [Tue, 22 Dec 2015 18:05:56 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for SfcClassifier"

8 years agoMerge "Use initFlowBuilder for SfcClassifierService"
Sam Hague [Tue, 22 Dec 2015 18:05:49 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for SfcClassifierService"

8 years agoMerge "Use initFlowBuilder for OF13Provider"
Sam Hague [Tue, 22 Dec 2015 18:05:41 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for OF13Provider"

8 years agoMerge "Use initFlowBuilder for IngressAclService"
Sam Hague [Tue, 22 Dec 2015 18:05:34 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for IngressAclService"

8 years agoMerge "Use initFlowBuilder for EgressAclService"
Sam Hague [Tue, 22 Dec 2015 18:05:24 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for EgressAclService"

8 years agoMerge "Use initFlowBuilder for RoutingService"
Sam Hague [Tue, 22 Dec 2015 18:05:13 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for RoutingService"

8 years agoMerge "Use initFlowBuilder for OutboundNatService"
Sam Hague [Tue, 22 Dec 2015 18:05:05 +0000 (18:05 +0000)]
Merge "Use initFlowBuilder for OutboundNatService"

8 years agoMerge "Use initFlowBuilder for LoadBalancerService"
Sam Hague [Tue, 22 Dec 2015 18:04:55 +0000 (18:04 +0000)]
Merge "Use initFlowBuilder for LoadBalancerService"

8 years agoMerge "Use initFlowBuilder for L3ForwardingService"
Sam Hague [Tue, 22 Dec 2015 18:04:46 +0000 (18:04 +0000)]
Merge "Use initFlowBuilder for L3ForwardingService"

8 years agoMerge "Use initFlowBuilder for InboundNatService"
Sam Hague [Tue, 22 Dec 2015 18:04:37 +0000 (18:04 +0000)]
Merge "Use initFlowBuilder for InboundNatService"

8 years agoMerge "Use initFlowBuilder for ClassifierService"
Sam Hague [Tue, 22 Dec 2015 18:04:30 +0000 (18:04 +0000)]
Merge "Use initFlowBuilder for ClassifierService"

8 years agoMerge "Use initFlowBuilder for ArpResponderService"
Sam Hague [Tue, 22 Dec 2015 18:04:20 +0000 (18:04 +0000)]
Merge "Use initFlowBuilder for ArpResponderService"

8 years agoMerge "Use strict for flows"
Sam Hague [Tue, 22 Dec 2015 18:03:26 +0000 (18:03 +0000)]
Merge "Use strict for flows"

8 years agoUse initFlowBuilder for SfcClassifier
Sam Hague [Tue, 22 Dec 2015 14:47:09 +0000 (09:47 -0500)]
Use initFlowBuilder for SfcClassifier

Change-Id: I06d398cac6faeaae76718e306159d0cfe6a55708
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse initFlowBuilder for SfcClassifierService
Sam Hague [Tue, 22 Dec 2015 14:39:57 +0000 (09:39 -0500)]
Use initFlowBuilder for SfcClassifierService

Change-Id: I88349b11efa11638930f90b4bf7232d36ad3d36e
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse initFlowBuilder for OF13Provider
Sam Hague [Tue, 22 Dec 2015 14:18:26 +0000 (09:18 -0500)]
Use initFlowBuilder for OF13Provider

Change-Id: I4ea3ed96bd99bc7cf1d3465513a209be76444544
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse initFlowBuilder for IngressAclService
Sam Hague [Tue, 22 Dec 2015 14:01:46 +0000 (09:01 -0500)]
Use initFlowBuilder for IngressAclService

Change-Id: Iec15b1d9d368cb715a7fa1aa5aaf8dc341c2f531
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse initFlowBuilder for EgressAclService
Sam Hague [Tue, 22 Dec 2015 02:49:13 +0000 (21:49 -0500)]
Use initFlowBuilder for EgressAclService

Change-Id: I96b5ba5b99421366eeebbe6a358b983fe248b2d1
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse ${project.version} instead of ovsdb.ui.version
Stephen Kitt [Tue, 22 Dec 2015 13:35:06 +0000 (14:35 +0100)]
Use ${project.version} instead of ovsdb.ui.version

This aligns ovsdb-ui with the rest of the project. The definition of
ovsdb.ui.version was removed in
I0a795d81e4422a5677981a1396ac36f90e103d35 but it was still referenced
in a couple of places.

Change-Id: Icd3a70c25aaf1af3429b5dfac488ad504de16fd2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
8 years agoUse initFlowBuilder for RoutingService
Sam Hague [Tue, 22 Dec 2015 02:10:06 +0000 (21:10 -0500)]
Use initFlowBuilder for RoutingService

Change-Id: Ib740544ddbbb08745aa7f21aa9246acad1ad1d53
Signed-off-by: Sam Hague <shague@redhat.com>
8 years agoUse initFlowBuilder for OutboundNatService
Sam Hague [Tue, 22 Dec 2015 01:52:50 +0000 (20:52 -0500)]
Use initFlowBuilder for OutboundNatService

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