ovsdb.git
9 years agoBug 1824 : OpenFlow13 Protocol plugin is made the default. Also Karaf edition doesnt... 84/11084/1
Madhu Venugopal [Fri, 12 Sep 2014 03:08:48 +0000 (20:08 -0700)]
Bug 1824 : OpenFlow13 Protocol plugin is made the default. Also Karaf edition doesnt support the legacy openflow protocol plugin.

Change-Id: I7ae3ea575f0c92a9da13176f3291a4f920d8cb34
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoBug 1822 - Adding the missing karaf distribution 72/11072/1
Madhu Venugopal [Thu, 11 Sep 2014 21:50:55 +0000 (14:50 -0700)]
Bug 1822 -  Adding the missing karaf distribution

Change-Id: I5c46bc70eb28f21420c6d441672fcec9cf23b503
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoBug 960 : Avoid adding redundant Openflow Controller entries 06/9606/8
Flavio Fernandes [Sat, 2 Aug 2014 04:20:58 +0000 (00:20 -0400)]
Bug 960 : Avoid adding redundant Openflow Controller entries

* Added IT for reproducing the problem;
* Enhanced ConfigurationService.setOFController() to detect duplicate
  targets and update rows instead of adding new ones.

Log: https://gist.github.com/anonymous/4ce9b2abdecda3c5e7fa

Change-Id: Ifedc351e849a18feb66b6025189495cb991c38f2
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "Bug 1799: Fixed broken Vagrantfile URL"
Dave Tucker [Thu, 11 Sep 2014 16:36:20 +0000 (16:36 +0000)]
Merge "Bug 1799: Fixed broken Vagrantfile URL"

9 years agoMerge "Bug 1811 Fixed the pipeline by moving L2Fwd to the end."
Dave Tucker [Thu, 11 Sep 2014 14:32:04 +0000 (14:32 +0000)]
Merge "Bug 1811 Fixed the pipeline by moving L2Fwd to the end."

9 years agoBug 1811 Fixed the pipeline by moving L2Fwd to the end. 49/11049/2
Madhu Venugopal [Thu, 11 Sep 2014 13:06:36 +0000 (06:06 -0700)]
Bug 1811 Fixed the pipeline by moving L2Fwd to the end.

Change-Id: I4192003c7b24d7bfb39abbfbcaf0772bb473a2a3
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoFix Devstack Vagrant VMs to use a later OS 47/11047/1
Dave Tucker [Thu, 11 Sep 2014 13:06:57 +0000 (14:06 +0100)]
Fix Devstack Vagrant VMs to use a later OS

Fixes bug 1798

Change-Id: Ibcd179e082c90e9933d822e190d9968a13e95bf4
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoBug 1799: Fixed broken Vagrantfile URL 15/11015/1
Daniel Farrell [Wed, 10 Sep 2014 21:25:31 +0000 (17:25 -0400)]
Bug 1799: Fixed broken Vagrantfile URL

Change-Id: Ibaad92043980d2be7ff0d7be10126df510367895
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
9 years agoMerge "Bug 1760 - Removing redudant call to writeFlow in L3ForwardingService"
Dave Tucker [Tue, 9 Sep 2014 12:43:33 +0000 (12:43 +0000)]
Merge "Bug 1760 - Removing redudant call to writeFlow in L3ForwardingService"

9 years agoBug 1760 - Removing redudant call to writeFlow in L3ForwardingService 30/10930/1
Madhu Venugopal [Tue, 9 Sep 2014 03:41:57 +0000 (20:41 -0700)]
Bug 1760 - Removing redudant call to writeFlow in L3ForwardingService

Change-Id: I5e95352ab5990e442b967fcd9845ea22b770afaf
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoCheck that a value is present before calling Optional.get() 26/10926/1
Dave Tucker [Mon, 8 Sep 2014 23:33:40 +0000 (00:33 +0100)]
Check that a value is present before calling Optional.get()

Fixes bug 1655

Change-Id: I6fdf8d5a0abab7af8bd95a2ebaaf68658c60d24e
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoBug 1461 - precondition in getProvider failing 21/10921/1
Madhu Venugopal [Mon, 8 Sep 2014 19:15:28 +0000 (12:15 -0700)]
Bug 1461 - precondition in getProvider failing

Since most of the Services that we support uses OpenFlow13 and also we are deprecating the use of Openflow10 plugin,
use the Openflow13 provider directly.

Change-Id: Ibecae733b0c735303908dacfce18e6f1d382e8f1
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoBug 1729: Make BlockingQueue events in EventDispatcherImpl volatile 63/10863/3
Flavio Fernandes [Fri, 5 Sep 2014 20:09:08 +0000 (16:09 -0400)]
Bug 1729: Make BlockingQueue events in EventDispatcherImpl volatile

Just like queue in PipelineOrchestratorImpl, events in EventDispatcherImpl
should be volatile since it is expected to be modified by different threads.

Ref gerrit in which Madhu fixes a similar issue in PipelineOrchestratorImpl:
https://git.opendaylight.org/gerrit/#/c/10843/

Change-Id: I923cb974788b0425e28af7823bf6cd23aeeb0f5f
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "Bug 1730: L3: Trigger processing of neutron port updates based on events from...
Madhu Venugopal [Sat, 6 Sep 2014 22:03:28 +0000 (22:03 +0000)]
Merge "Bug 1730: L3: Trigger processing of neutron port updates based on events from southbound"

9 years agoBug 1730: L3: Trigger processing of neutron port updates based on events from southbound 62/10862/2
Flavio Fernandes [Fri, 5 Sep 2014 19:56:12 +0000 (15:56 -0400)]
Bug 1730: L3: Trigger processing of neutron port updates based on events from southbound

When interface events originate from the southbound side of ovsdb, the L3 forwarding
feature in ovsdb expects to be notified. W/out this code, flows needed to program L3
functionality will get skipped.

Due to my mistake, the logic for connecting the southbound event to the
already implemented neutron port handler is missing. This bug is created
to account for the the small code missing in L3 handler; past the code
freeze.

Change-Id: I7d124a66425352dd4690ab830cc29d0b94a87e67
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "Adding SFC feature dependencies to ovs-sfc module"
Brent Salisbury [Fri, 5 Sep 2014 14:57:08 +0000 (14:57 +0000)]
Merge "Adding SFC feature dependencies to ovs-sfc module"

9 years agoAdding SFC feature dependencies to ovs-sfc module 55/10855/1
Madhu Venugopal [Fri, 5 Sep 2014 14:44:27 +0000 (07:44 -0700)]
Adding SFC feature dependencies to ovs-sfc module

Change-Id: I5067febf5d1d3a006abc6d5caf19df31102bc99a
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "With the unrelated karaf dependency issues not seen (for now) reenabling karaf...
Dave Tucker [Fri, 5 Sep 2014 12:30:34 +0000 (12:30 +0000)]
Merge "With the unrelated karaf dependency issues not seen (for now) reenabling karaf builds in jenkins"

9 years agoMerge "Moving the InventoryListener out of AbstractServiceInstance and into a common...
Dave Tucker [Fri, 5 Sep 2014 12:30:16 +0000 (12:30 +0000)]
Merge "Moving the InventoryListener out of AbstractServiceInstance and into a common place PipelineOrchestratorImpl"

9 years agoMoving the InventoryListener out of AbstractServiceInstance and into a common place... 43/10843/3
Madhu Venugopal [Fri, 5 Sep 2014 10:27:11 +0000 (03:27 -0700)]
Moving the InventoryListener out of AbstractServiceInstance and into a common place PipelineOrchestratorImpl

Change-Id: Ic443bfcb160122f9e5e943aa64c36a73034a6c88
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoWith the unrelated karaf dependency issues not seen (for now) reenabling karaf builds... 44/10844/1
Madhu Venugopal [Fri, 5 Sep 2014 11:07:14 +0000 (04:07 -0700)]
With the unrelated karaf dependency issues not seen (for now) reenabling karaf builds in jenkins

Change-Id: Ib1389b03a530fe7250246c0405b0cd010496e124
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Combine ovssfc patches."
Madhu Venugopal [Fri, 5 Sep 2014 09:48:22 +0000 (09:48 +0000)]
Merge "Combine ovssfc patches."

9 years agoInMemoryDOMDataStore$DOMStoreTransactionChainImpl isnt happy if there are mulitple... 16/10816/1
Madhu Venugopal [Thu, 4 Sep 2014 23:13:10 +0000 (16:13 -0700)]
InMemoryDOMDataStore$DOMStoreTransactionChainImpl isnt happy if there are mulitple transactions queued in quick succession.

And it throws
java.lang.IllegalStateException: Previous transaction in chain must be ready.
at com.google.common.base.Preconditions.checkState(Preconditions.java:149) ~[na:na]
at org.opendaylight.controller.md.sal.dom.store.impl.InMemoryDOMDataStore$DOMStoreTransactionChainImpl.newReadWriteTransaction

when we reuse the txchain. Fixing it temporarily. Better solution is to get away from this and use DataChangeListener.

Change-Id: Id20abdefc21574dce9ff078f4ce96aa91c390edb
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Added LoadBalancerProvider as the service provided by LoadBalancerService"
Brent Salisbury [Thu, 4 Sep 2014 21:45:40 +0000 (21:45 +0000)]
Merge "Added LoadBalancerProvider as the service provided by LoadBalancerService"

9 years agoMerge "Splitting OF13Provider's native L2 Forwarding code into Classifier and L2Forwa...
Brent Salisbury [Thu, 4 Sep 2014 21:43:10 +0000 (21:43 +0000)]
Merge "Splitting OF13Provider's native L2 Forwarding code into Classifier and L2Forwarding Table in the pipeline"

9 years agoMerge "L3: Use system config to get the mac address of the default gateway in the...
Madhu Venugopal [Thu, 4 Sep 2014 21:38:34 +0000 (21:38 +0000)]
Merge "L3: Use system config to get the mac address of the default gateway in the external network."

9 years agoAdded LoadBalancerProvider as the service provided by LoadBalancerService 12/10812/1
Madhu Venugopal [Thu, 4 Sep 2014 19:12:40 +0000 (12:12 -0700)]
Added LoadBalancerProvider as the service provided by LoadBalancerService

Change-Id: I8168f0490d8b9cf5b477d177cfa5197f41deb3b6
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoL3: Remove redundant line added in commit e38b7ff6 01/10801/1
Flavio Fernandes [Thu, 4 Sep 2014 18:31:01 +0000 (14:31 -0400)]
L3: Remove redundant line added in commit e38b7ff6

Change-Id: If90b95464dd1c022fd1ed74868c01de050fb89b0
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoL3: Use system config to get the mac address of the default gateway in the external... 92/10792/1
Flavio Fernandes [Thu, 4 Sep 2014 16:53:57 +0000 (12:53 -0400)]
L3: Use system config to get the mac address of the default gateway in the external network.

Change-Id: Iab1d50af3c42a0b2552aa378d8c5f59dc9e3914a
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "L3: Add support for distributed arp responder and l3 forwarding"
Dave Tucker [Thu, 4 Sep 2014 12:53:20 +0000 (12:53 +0000)]
Merge "L3: Add support for distributed arp responder and l3 forwarding"

9 years agoCombine ovssfc patches. 72/10772/1
Sam Hague [Thu, 4 Sep 2014 12:37:21 +0000 (08:37 -0400)]
Combine ovssfc patches.

Change-Id: I9d0e6f29c8ea837ceccec1073fea549ad77a2b3f
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoSplitting OF13Provider's native L2 Forwarding code into Classifier and L2Forwarding... 71/10771/1
Madhu Venugopal [Thu, 4 Sep 2014 12:38:31 +0000 (05:38 -0700)]
Splitting OF13Provider's native L2 Forwarding code into Classifier and L2Forwarding Table in the pipeline

There are a few issues with the Flooding rules wherein Tunnel vs VM port Flooding / Broadcast flows step
on each-other instead of work together. Will file bugs to get them resolved.

Change-Id: I47706ce0bd7482eb76e48a28bedad31f434aa5af
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoSplit Ingress and Egress ACL Programming 56/10756/1
Dave Tucker [Thu, 4 Sep 2014 09:18:05 +0000 (10:18 +0100)]
Split Ingress and Egress ACL Programming

Change-Id: I7ccb0f7cfced6410efa95cc82dd6e51e8739fbe9
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoAdd ovssfc as it's own karafe feature within ovsdb 44/10744/2
Sam Hague [Thu, 4 Sep 2014 01:43:04 +0000 (21:43 -0400)]
Add ovssfc as it's own karafe feature within ovsdb

Change-Id: Ib1f2b26c255f9d86e876d2ada0f875f73ce16a5e
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "ACL services for OS/OVS"
Madhu Venugopal [Thu, 4 Sep 2014 01:31:31 +0000 (01:31 +0000)]
Merge "ACL services for OS/OVS"

9 years agoACL services for OS/OVS 65/10565/7
Brent Salisbury [Mon, 1 Sep 2014 01:41:31 +0000 (21:41 -0400)]
ACL services for OS/OVS

Patch #1:Ingress ACLs
Patch #2:Egress ACLs
Patch #3:Rebase and typos
Patch #4:Service exports patching
Patch #5:Added IngressACL Interface and verified flowmods
Patch #6:Deleted random .diff file in last commit

TODO:
-Break egress out of the IngressACLService into EgressACLService
Easier to tshoot colapsed.
-CRUD updates and deletes.
-Refactor GOTO instructs w/the pipeline mutator fromt the abstract
services parent for Egress ACL (completed for Ingress)

Change-Id: I2a7bea87e7d75b6b02c8ff6a43b60f881c4ade02
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>
9 years agoNorthboundV3 APIs for RowResource 35/10735/1
Madhu Venugopal [Wed, 3 Sep 2014 19:58:12 +0000 (12:58 -0700)]
NorthboundV3 APIs for RowResource

Change-Id: I9242e655c88424896ded0b8c5a171cdc6df22705
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoL3: Add support for distributed arp responder and l3 forwarding 31/10731/1
Flavio Fernandes [Wed, 3 Sep 2014 20:56:46 +0000 (16:56 -0400)]
L3: Add support for distributed arp responder and l3 forwarding

Test log: https://gist.github.com/f3795b3163b778860048

Change-Id: Id00d64d9e62e97d0da082589f37a9470847b395d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoNorthboundV3 APIs for TableResource 23/10723/2
Madhu Venugopal [Wed, 3 Sep 2014 18:34:42 +0000 (11:34 -0700)]
NorthboundV3 APIs for TableResource

Change-Id: I59187c4eb737523f15f47402f1bb3d757223f894
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoNorthboundV3 APIs for DatabaseResource. 22/10722/2
Madhu Venugopal [Wed, 3 Sep 2014 18:31:29 +0000 (11:31 -0700)]
NorthboundV3 APIs for DatabaseResource.

Change-Id: I8d3cb65ac101b8f9898c621c8a7b897f17e58d3b
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoNorthboundV3 APIs for NodeResource. Supporting just the GET APIs. 21/10721/2
Madhu Venugopal [Wed, 3 Sep 2014 18:27:52 +0000 (11:27 -0700)]
NorthboundV3 APIs for NodeResource. Supporting just the GET APIs.

Change-Id: I5f3cd6b8b578ad975d0b2d941af99a34ce86ad63
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoNorthboundV3 insert Row using the newly added insertTree plugin functionality 20/10720/2
Madhu Venugopal [Wed, 3 Sep 2014 12:41:32 +0000 (05:41 -0700)]
NorthboundV3 insert Row using the newly added insertTree plugin functionality

Change-Id: I7a32798510b0f835ff9aed8e8a0da301188f7895
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoImplemented Schema independent Plugin Read, Update and Delete APIs 11/10611/3
Madhu Venugopal [Tue, 2 Sep 2014 01:44:09 +0000 (18:44 -0700)]
Implemented Schema independent Plugin Read, Update and Delete APIs

Change-Id: I92f2178df30c2fe5fc69166a98aa08d2da86663f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Updating the LB pool member DELETE and UPDATE processing."
Madhu Venugopal [Wed, 3 Sep 2014 11:01:30 +0000 (11:01 +0000)]
Merge "Updating the LB pool member DELETE and UPDATE processing."

9 years agoMerge "Adding support for programming newly added nodes."
Madhu Venugopal [Wed, 3 Sep 2014 11:01:18 +0000 (11:01 +0000)]
Merge "Adding support for programming newly added nodes."

9 years agoMerge "In LoadBalancerProvider interface, rename method programLoadBalancerMemberRule...
Madhu Venugopal [Wed, 3 Sep 2014 10:48:49 +0000 (10:48 +0000)]
Merge "In LoadBalancerProvider interface, rename method programLoadBalancerMemberRules to programLoadBalancerPoolMemberRules, for consistency sakes."

9 years agoMerge "Net-Virt LBaaS module updates for neutron event handling"
Madhu Venugopal [Wed, 3 Sep 2014 10:48:31 +0000 (10:48 +0000)]
Merge "Net-Virt LBaaS module updates for neutron event handling"

9 years agoMerge "Restructing the LBaaSHandler code, and adding calls to the LoadBalancerProvide...
Madhu Venugopal [Wed, 3 Sep 2014 10:44:02 +0000 (10:44 +0000)]
Merge "Restructing the LBaaSHandler code, and adding calls to the LoadBalancerProvider for the ADD and DELETE operations."

9 years agoMerge "L3: Break exclusions and rewrites into inbound and outbound conterparts."
Dave Tucker [Wed, 3 Sep 2014 09:55:20 +0000 (09:55 +0000)]
Merge "L3: Break exclusions and rewrites into inbound and outbound conterparts."

9 years agoUpdating the LB pool member DELETE and UPDATE processing. 74/10674/1
Srini Seetharaman [Wed, 3 Sep 2014 08:05:28 +0000 (01:05 -0700)]
Updating the LB pool member DELETE and UPDATE processing.

- DELETE will lead to enqueuing an UPDATE LB instance event
- UPDATE will be ignored because weight update is not supported.

Change-Id: I05d192956bc16bb124fa4cc5a7dcf035f63bc9a1
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoAdding support for programming newly added nodes. 72/10672/1
Srini Seetharaman [Wed, 3 Sep 2014 07:49:04 +0000 (00:49 -0700)]
Adding support for programming newly added nodes.

LBaaSHandler now implements IInventoryListener, and
programs all existing LB instances on the newly added node.

Change-Id: I0544f7d30f422464440582e98da138157fc8ffd6
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoIn LoadBalancerProvider interface, rename method programLoadBalancerMemberRules 71/10671/1
Srini Seetharaman [Wed, 3 Sep 2014 07:24:30 +0000 (00:24 -0700)]
In LoadBalancerProvider interface, rename method programLoadBalancerMemberRules
to programLoadBalancerPoolMemberRules, for consistency sakes.

Change-Id: Id10296362b186a1b76bbbacd6fa4ff9b65bc2bed
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoNet-Virt LBaaS module updates for neutron event handling 70/10670/1
Srini Seetharaman [Wed, 3 Sep 2014 07:18:05 +0000 (00:18 -0700)]
Net-Virt LBaaS module updates for neutron event handling

- Adding LBaaSPoolMemberHandler for handling pool member ADD/DELETE operations.
- Moving out cache processing utility to NeutronCacheUtils
- Updating variable names in LBaaSHandler to maintain consistency.

Change-Id: I664e7d85bd1cc6e2dbe9f0b51fd486eb115d0bd9
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoRestructing the LBaaSHandler code, and adding calls to the 66/10666/2
Srini Seetharaman [Wed, 3 Sep 2014 05:03:46 +0000 (22:03 -0700)]
Restructing the LBaaSHandler code, and adding calls to the
LoadBalancerProvider for the ADD and DELETE operations.

Change-Id: I2b7991abe8c2d5a1abad74cc510a576d67a98219
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoAdding logic and code for pushing FlowMods in the LoadBalancerService. 64/10664/1
Srini Seetharaman [Wed, 3 Sep 2014 04:00:37 +0000 (21:00 -0700)]
Adding logic and code for pushing FlowMods in the LoadBalancerService.

The current implementation supports ADD for LB instance and also ADD
for LB member. DELETE for LB member is unsupported because of need to
refresh the multipath indices. TODO: FlowMods for DELETE LB instance.

Change-Id: Ifcd2f95a67168c89d398abb9f43a3a76e0609589
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoMerge "Making changes to LBaaSHandler and Activator to add logic for LB Neutron calls."
Madhu Venugopal [Wed, 3 Sep 2014 03:55:02 +0000 (03:55 +0000)]
Merge "Making changes to LBaaSHandler and Activator to add logic for LB Neutron calls."

9 years agoMerge "Splitting the LBaaS northbound events into three individual event types (speci...
Madhu Venugopal [Wed, 3 Sep 2014 03:49:17 +0000 (03:49 +0000)]
Merge "Splitting the LBaaS northbound events into three individual event types (specifically, LB, LB-Pool, LB-Pool-Member)"

9 years agoL3: Break exclusions and rewrites into inbound and outbound conterparts. 43/10643/3
Flavio Fernandes [Tue, 2 Sep 2014 16:49:12 +0000 (12:49 -0400)]
L3: Break exclusions and rewrites into inbound and outbound conterparts.

patch 2: remove todo

Change-Id: I57a26bf5f8b40daa19939beecc27f55d8ae3d70c
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoAdd Flow Programming for L3 Services 56/10656/2
Dave Tucker [Tue, 2 Sep 2014 21:51:35 +0000 (22:51 +0100)]
Add Flow Programming for L3 Services

This commit adds programming of L3 services.
In addition, we add a deleteFlow method to the AbstractServiceInstance
class

Change-Id: I4d9fe8c93e6a93732660f3895df7885cda015b7e
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoMaking changes to LBaaSHandler and Activator to add logic for LB Neutron calls. 52/10652/3
Srini Seetharaman [Tue, 2 Sep 2014 19:32:55 +0000 (12:32 -0700)]
Making changes to LBaaSHandler and Activator to add logic for LB Neutron calls.

- Cleaning up LBaaSHandler class to only handle the NEUTRON_LOAD_BALANCER event
- Cleaning up LBaaSHandler to only be INeutronLoadBalancerAware
- Adding the service LoadBalancerProvider and adding dependency in Activator
- Adding logic to handle the neutron LB instance creation call. After the call,
  the information will be stored in a Map <UUID, LoadBalancerConfiguration>
  object. Still TODO: FlowMods through the LoadBalancerService.

Change-Id: Ic20d10ea680730de59bbe60a12c47fc6684a6bab
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoSplitting the LBaaS northbound events into three individual event types 51/10651/3
Srini Seetharaman [Tue, 2 Sep 2014 19:27:56 +0000 (12:27 -0700)]
Splitting the LBaaS northbound events into three individual event types
(specifically, LB, LB-Pool, LB-Pool-Member)

Change-Id: I3a9604ecdfeff018a720514adecec1ffd5824c39
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoSkip Karaf Tests By Default 05/10405/3
Dave Tucker [Wed, 27 Aug 2014 23:48:04 +0000 (00:48 +0100)]
Skip Karaf Tests By Default

Karaf tests take far too long to run therefore we will skip by default.
OVSDB will create a daily job to check Karaf stability

Change-Id: I8bdb6b6f870956a78e45fa3973085a50f882126f
Signed-off-by: Dave Tucker <djt@redhat.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoSkeleton implementation of L3 using PipelineOrchestration 78/10578/5
Dave Tucker [Mon, 1 Sep 2014 14:42:43 +0000 (15:42 +0100)]
Skeleton implementation of L3 using PipelineOrchestration

This refactors the L3 interface to fit the static pipeline model

L3 logic stays in net-virt and the service tables in net-virt-providers
implement the new interfaces. The logic in net-virt can resolve the
implementation of these interfaces through the OSGi SR.

Ideally service lookup should not be handled in the net-virt Activator,
instead we should allow the OFXProvider classes to handle this by
watching for implementations of AbstractServiceInstance with a service
property that matches their name.

Change-Id: I0e41d992e0ff1d8de2d25b02ce8e5a36e27e6ae7
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoMerge "L3 Neutron events hookup to MultiTenantRouterForwardingProvider"
Dave Tucker [Tue, 2 Sep 2014 10:51:39 +0000 (10:51 +0000)]
Merge "L3 Neutron events hookup to MultiTenantRouterForwardingProvider"

9 years agoIn AbstractServiceInstance, changing return value of getTable() 10/10610/1
Srini Seetharaman [Tue, 2 Sep 2014 02:06:38 +0000 (19:06 -0700)]
In AbstractServiceInstance, changing return value of getTable()
to short, and making writeFlow() protected.

Change-Id: I28ba2899c77bf188be6a25f3d590f90b7ce7de9a
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoMaking changes to nxMultipathAction with MD-SAL 09/10609/1
Srini Seetharaman [Tue, 2 Sep 2014 02:04:42 +0000 (19:04 -0700)]
Making changes to nxMultipathAction with MD-SAL

- Adding missing "dst" field in Yang nx-action-multipath-grouping
- Adding missing field to the ActionUtils.nxMultipathAction
- Adding setNwSrcAction and setNwDstAction to ActionUtils

Change-Id: Ie0aab975b17620a2a2b74b7a69acfd9251ddc6e4
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoEnhanced Schema indepenent plugin APIs by supporting optional parent_table and parent... 99/10599/1
Madhu Venugopal [Mon, 1 Sep 2014 18:54:42 +0000 (11:54 -0700)]
Enhanced Schema indepenent plugin APIs by supporting optional parent_table and parent_column for insertRow / insertTree

Change-Id: Id0f0fb6183c5f39e2b524dc9a7ef8cf699d9ef76
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Added Template classes for all the static services."
Brent Salisbury [Sun, 31 Aug 2014 21:52:34 +0000 (21:52 +0000)]
Merge "Added Template classes for all the static services."

9 years agoMerge "AbstractServiceInstance is fully self-suffient to program the default flow...
Brent Salisbury [Sun, 31 Aug 2014 21:52:15 +0000 (21:52 +0000)]
Merge "AbstractServiceInstance is fully self-suffient to program the default flow-rules for any Service in the pipeline."

9 years agoMerge "Static Pipeline is constructed with the help of AbstractServiceInstance and...
Brent Salisbury [Sun, 31 Aug 2014 21:51:56 +0000 (21:51 +0000)]
Merge "Static Pipeline is constructed with the help of AbstractServiceInstance and PipelineOrchestrator."

9 years agoMerge "Changed the current WIP Pipeline Orchestrator to a more simplified Static...
Brent Salisbury [Sun, 31 Aug 2014 21:51:23 +0000 (21:51 +0000)]
Merge "Changed the current WIP Pipeline Orchestrator to a more simplified Static Pipeline that is easy to understand and debug than the dynamic pipeline management."

9 years agoMerge "Moved OF10Provider and OF13Provider to its own Package to accomodate the upcom...
Brent Salisbury [Sun, 31 Aug 2014 21:50:58 +0000 (21:50 +0000)]
Merge "Moved OF10Provider and OF13Provider to its own Package to accomodate the upcoming Pipeline changes"

9 years agoMerge "WIP: Initial Commit of Pipeline Orchestrator"
Madhu Venugopal [Sun, 31 Aug 2014 21:32:41 +0000 (21:32 +0000)]
Merge "WIP: Initial Commit of Pipeline Orchestrator"

9 years agoAdded Template classes for all the static services. 54/10554/2
Madhu Venugopal [Sun, 31 Aug 2014 19:30:54 +0000 (12:30 -0700)]
Added Template classes for all the static services.

Though these are just Template classes, Since it inherits from AbstractServiceInstance parent, the default catch-all flow-rules
(that is required for an operational pipeline) are automatically installed. All the service specific code can be written in these
classes with the guarantee that the parent AbstractServiceInstance took care of the flows to keep the pipeline happy.

Also, please note that, all of these services must use the getMutablePipelineInstructionBuilder() method for all the non-drop
matched flows to make sure the pipeline is not messed up for the cases where a service performs some operations.

Sample output of the pipeline when none of the services are operational :

[mininet@opendaylight ~]\>sudo ovs-ofctl dump-flows -O OpenFlow13 br-int
OFPST_FLOW reply (OF1.3) (xid=0x2):
 cookie=0x0, duration=136.403s, table=0, n_packets=0, n_bytes=0, send_flow_rem dl_type=0x88cc actions=CONTROLLER:65535
 cookie=0x0, duration=136.418s, table=0, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:20
 cookie=0x0, duration=136.436s, table=20, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:30
 cookie=0x0, duration=136.427s, table=30, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:40
 cookie=0x0, duration=136.429s, table=40, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:50
 cookie=0x0, duration=136.421s, table=50, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:60
 cookie=0x0, duration=136.424s, table=60, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:70
 cookie=0x0, duration=138.24s, table=70, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:80
 cookie=0x0, duration=136.4s, table=80, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:90
 cookie=0x0, duration=136.433s, table=90, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=goto_table:100
 cookie=0x0, duration=136.441s, table=100, n_packets=3, n_bytes=230, send_flow_rem priority=0 actions=drop

Please note that, since we have moved to a static pipeline approach, there is no need for metadata, nor there is a
need to have a dedicated Director table. The AbstractServiceInstance takes care of cooking the pipeline based on the
static pipeline requirements.

Change-Id: Icf712991df3c7fbedf81872c04fdad168619b910
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoAbstractServiceInstance is fully self-suffient to program the default flow-rules... 51/10551/2
Madhu Venugopal [Sun, 31 Aug 2014 18:10:50 +0000 (11:10 -0700)]
AbstractServiceInstance is fully self-suffient to program the default flow-rules for any Service in the pipeline.

Changes includes :
- Register as Notification listener with the MD-SAL data-store.
- Process the NodeUpdate and identify openflow nodes before submitting for further processing
- Process Notifications in its own Thread
- Program the default catch-all rule

Change-Id: I302dbcd95de29cc2d7ff55791305f36a8f07456b
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoStatic Pipeline is constructed with the help of AbstractServiceInstance and PipelineO... 46/10546/1
Madhu Venugopal [Sun, 31 Aug 2014 14:04:05 +0000 (07:04 -0700)]
Static Pipeline is constructed with the help of AbstractServiceInstance and PipelineOrchestrator.

- Since we decided to implement the Static pipeline, there is no datapath requirement to keep a Director.
  Since the pipeline is static, the PipelineOrchestrator coordinages with the AbstractServiceInstance to program the direct path between the Tables
  without any involvement from the Concrete Service that is implemented on top of AbstractServiceInstance.
  Hence, this change removes Director table from the pipeline. But, the software design remains intact wherein the logic of pipeline datapath management
  still resides in AbstractServiceInstance, which the Concrete Service class need not be worried about.
- Implemented a sample L2RewriteService that extends from AbstractServiceInstance and registers itself with PipelineOrchestratorImpl through regular OSGi Service Registry

Change-Id: Ib27a795ebd428fc0a3b97aa3d18d825fe8427246
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoChanged the current WIP Pipeline Orchestrator to a more simplified Static Pipeline... 35/10535/1
Madhu Venugopal [Sun, 31 Aug 2014 03:52:14 +0000 (20:52 -0700)]
Changed the current WIP Pipeline Orchestrator to a more simplified Static Pipeline that is easy to understand and debug than the dynamic pipeline management.

Change-Id: Id53b7a814a433c450fd9780c0025f0f3e9ec9473
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMoved OF10Provider and OF13Provider to its own Package to accomodate the upcoming... 34/10534/1
Madhu Venugopal [Sun, 31 Aug 2014 02:00:21 +0000 (19:00 -0700)]
Moved OF10Provider and OF13Provider to its own Package to accomodate the upcoming Pipeline changes

Change-Id: Ia998c272372ed559bcab561e21d7864de03533b0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Fixed a minor bug that causes the deprecated CSIT Integration Bridge Domain...
Brent Salisbury [Sun, 31 Aug 2014 00:53:43 +0000 (00:53 +0000)]
Merge "Fixed a minor bug that causes the deprecated CSIT Integration Bridge Domain addPort tests to fail for a long time."

9 years agoFixed a minor bug that causes the deprecated CSIT Integration Bridge Domain addPort... 19/10519/2
Madhu Venugopal [Sat, 30 Aug 2014 16:40:25 +0000 (09:40 -0700)]
Fixed a minor bug that causes the deprecated CSIT Integration Bridge Domain addPort tests to fail for a long time.

Change-Id: If17a4de8d0832c31de4fb8d5ee38cfe353c0d326
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoL3 Neutron events hookup to MultiTenantRouterForwardingProvider 00/10500/2
Flavio Fernandes [Fri, 29 Aug 2014 02:20:30 +0000 (22:20 -0400)]
L3 Neutron events hookup to MultiTenantRouterForwardingProvider

Neutron Router interface and floating ips will now trigger calls into MultiTenantRouterForwardingProvider.

Added implementations:
 * programStaticArpEntry (router interface)
 * programIpRewriteExclusion
 * programRouterInterface
 * programDefaultRouteEntry
 * programStaticArpEntry (tenant vms)
 * programIpRewriteRule

        Script for creating coke tenants in devstack:
        https://gist.github.com/1ba897e7cde8dbd87fe8

        Starting odl command:
        ./run.sh -virt ovsdb -Xms40m -Xmx1024m -XX:MaxPermSize=1024m

        LogLevel:
        setLogLevel org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter trace

        Log of test:
        createTenants: https://gist.github.com/40364d727c40570fb4ff
        osgi: https://gist.github.com/105938470c9a19bebc4a

Change-Id: Ie4b30b2581c469064ca29a443d9ced4c2be4d3a9
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoWIP: Initial Commit of Pipeline Orchestrator 88/10388/2
Dave Tucker [Wed, 27 Aug 2014 13:27:41 +0000 (14:27 +0100)]
WIP: Initial Commit of Pipeline Orchestrator

Change-Id: Iee82bf6fc14dbffcea1af44a06cbb44f4bbd1706
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoMinor tweaks to interface used by Openstack L3 forwarding 83/10483/1
Flavio Fernandes [Fri, 29 Aug 2014 13:44:05 +0000 (09:44 -0400)]
Minor tweaks to interface used by Openstack L3 forwarding

Change-Id: Ie2c3e833027cee5e2d89c5323890c09f1969d42a
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "L3 Neutron plumbing"
Dave Tucker [Fri, 29 Aug 2014 13:04:19 +0000 (13:04 +0000)]
Merge "L3 Neutron plumbing"

9 years agoFixed dependencies for features-ovsdb. 75/10475/1
Hideyuki Tai [Fri, 29 Aug 2014 02:43:28 +0000 (22:43 -0400)]
Fixed dependencies for features-ovsdb.

Change-Id: I02ec0c02b66bc53e1463b73c81f6844054ac883f
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>
9 years agoL3 Neutron plumbing 63/10463/1
Flavio Fernandes [Thu, 28 Aug 2014 06:51:48 +0000 (02:51 -0400)]
L3 Neutron plumbing

Converge neutron events needed to support l3 into an adapter class.
With this change all plumbing needed for northbound l3 support is completed.
Next is the 'brains' to turn the l3 related events into rules that can be used
by the multi-tenant router forwarding provider.

    Script for creating coke tenants in devstack:
    https://gist.github.com/1ba897e7cde8dbd87fe8

    Starting odl command:
    ./run.sh -virt ovsdb -Xms40m -Xmx1024m -XX:MaxPermSize=1024m

    LogLevel:
    setLogLevel org.opendaylight.ovsdb.openstack.netvirt.providers.OF13Provider trace
    setLogLevel org.opendaylight.ovsdb.openstack.netvirt.api.EventDispatcher trace
    setLogLevel org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler debug
    setLogLevel org.opendaylight.ovsdb.openstack.netvirt.PortHandler debug
    setLogLevel org.opendaylight.ovsdb.openstack.netvirt.NeutronL3Adapter debug

    Log of test:
    createTenants: https://gist.github.com/7dadd30bde5d07bb0ba1
    osgi: https://gist.github.com/4ec66a6e76908eeffcbc

Change-Id: I1570d536728ea29c2615ddca422379ad8fe70d45
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoMerge "Add node and dpid information to L3 Forwarding API"
Madhu Venugopal [Thu, 28 Aug 2014 15:31:42 +0000 (15:31 +0000)]
Merge "Add node and dpid information to L3 Forwarding API"

9 years agoAdd node and dpid information to L3 Forwarding API 56/10456/1
Dave Tucker [Thu, 28 Aug 2014 14:56:59 +0000 (15:56 +0100)]
Add node and dpid information to L3 Forwarding API

Change-Id: I06186d17b3a26e262caee612d9221994adc8ca1c
Signed-off-by: Dave Tucker <djt@redhat.com>
9 years agoAdd ovssfc to osdvb distribution 07/10407/3
Sam Hague [Wed, 27 Aug 2014 23:49:26 +0000 (19:49 -0400)]
Add ovssfc to osdvb distribution

Change-Id: Ibcbf2783d1125fbf08314feb210ef7177165b96a
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoMerge "Karaf Feature support for OVS based Openstack Network Virtualization"
Brent Salisbury [Thu, 28 Aug 2014 07:33:52 +0000 (07:33 +0000)]
Merge "Karaf Feature support for OVS based Openstack Network Virtualization"

9 years agoKaraf Feature support for OVS based Openstack Network Virtualization 27/10427/1
Madhu Venugopal [Thu, 28 Aug 2014 07:24:17 +0000 (00:24 -0700)]
Karaf Feature support for OVS based Openstack Network Virtualization

Change-Id: I3becfbdd0141bf61be4296785568efdcccfc8f2b
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years ago- Adding LBaaS handler skeleton to net-virt bundle, appropriate entries 26/10426/1
Srini Seetharaman [Thu, 28 Aug 2014 06:59:33 +0000 (23:59 -0700)]
- Adding LBaaS handler skeleton to net-virt bundle, appropriate entries
in AbstractEvent and Activator.

- Fixing Postman examples for LBaaS.

Change-Id: Id878ffe5bf1e8b1819a18f103b5ee9c4fe74e05e
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoMerge "Adding Karaf feature for openflow Nicira Extensions support"
Brent Salisbury [Thu, 28 Aug 2014 06:16:10 +0000 (06:16 +0000)]
Merge "Adding Karaf feature for openflow Nicira Extensions support"

9 years agoMerge "Moved ovsdb features into a sub-directory to accomodate related features to...
Brent Salisbury [Thu, 28 Aug 2014 06:15:53 +0000 (06:15 +0000)]
Merge "Moved ovsdb features into a sub-directory to accomodate related features to come"

9 years agoAdding Karaf feature for openflow Nicira Extensions support 24/10424/1
Madhu Venugopal [Thu, 28 Aug 2014 05:59:20 +0000 (22:59 -0700)]
Adding Karaf feature for openflow Nicira Extensions support

Change-Id: I871c4c1d242770dd9b2f8a21b50f874388eeeb45
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMoved ovsdb features into a sub-directory to accomodate related features to come 23/10423/1
Madhu Venugopal [Thu, 28 Aug 2014 03:46:11 +0000 (20:46 -0700)]
Moved ovsdb features into a sub-directory to accomodate related features to come

Change-Id: I1ba0294faef385bbd94e180e102be5542e36afd9
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoFWaaS handler/callbacks for the Neutron service 50/10050/3
Brent Salisbury [Tue, 19 Aug 2014 06:24:17 +0000 (02:24 -0400)]
FWaaS handler/callbacks for the Neutron service

Patch #3: Activator tweek

Patch #2: Rebased and applied Flavio's patch. Thanks.

Patch #1: Plumbing the Controller OS FWaaS API into
the ovsdb net-virt bundle.

Change-Id: If037b03e2d954192e9721d67f45f31eb5d9fe66e
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>
9 years agoAdding Nicira action NXAST_MULTIPATH, appropriate SAL connectors, 54/10354/2
Srini Seetharaman [Wed, 27 Aug 2014 06:46:46 +0000 (23:46 -0700)]
Adding Nicira action NXAST_MULTIPATH, appropriate SAL connectors,
and utils.

Change-Id: Ia241fe5a728cf8f55449628a36a5bd2b160c7f61
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
9 years agoAdding plugin.shell to make OVSDB Karaf feature happy in integration branch 06/10406/1
Madhu Venugopal [Wed, 27 Aug 2014 23:55:20 +0000 (16:55 -0700)]
Adding plugin.shell to make OVSDB Karaf feature happy in integration branch

Change-Id: Id583df6317e56f5377aa51dbc32c7aed79b3905f
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
9 years agoMerge "Adjusted OVSDB features to enable the Pax-exam tests"
Dave Tucker [Wed, 27 Aug 2014 23:32:35 +0000 (23:32 +0000)]
Merge "Adjusted OVSDB features to enable the Pax-exam tests"