Dave Tucker [Mon, 22 Sep 2014 19:05:10 +0000 (20:05 +0100)]
Add L3 Forwarding Table to Pipeline
Fixes bug 2033
Change-Id: Id1fdb87ea617d83d3d8c035993aa3b9abcbfb258
Signed-off-by: Dave Tucker <djt@redhat.com>
Dave Tucker [Mon, 22 Sep 2014 16:37:36 +0000 (17:37 +0100)]
Only use single Apply Actions instruction
There should only be one Apply Actions instruction in an Instruction Set
Fixes bug 2029
Change-Id: I415a00c36e951b4b228cde60633b6361df23f77d
Signed-off-by: Dave Tucker <djt@redhat.com>
Madhu Venugopal [Mon, 22 Sep 2014 06:07:55 +0000 (23:07 -0700)]
Bug 1695 - ignoring pipeline rules for any non-forwarding bridge
By design, the AbstractServiceInstance kept the isBridgeInPipeline as an abstract method for its child concrete Services
to determine the behaviour.
But, the most important and basic case of allowing the pipeline flows on br-int and ignoring on all other Bridge seems
reasonable default in AbstractServiceInstance. This can be overriden by child Services if need be.
In addition to this fix, have also introduced a minor delay in PipelineOrchestrator as soon as a Node event is observed.
This should workaround one of the basic bugs (1997) on the pipeline management case at the minimum.
Change-Id: I34f1d24d3f72695851bda4379483c8ea2b292308
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Srini Seetharaman [Mon, 22 Sep 2014 06:22:52 +0000 (23:22 -0700)]
Bug 2004 - Fixing issue where local ports are not receivers for flood originating from another local port
The fix was to update classifier to track whether a flood is coming from local or
remote, and then matching on it to apply the enumerated flood action. The fix also
required OF13Provider to register local ports with the flood rule creation logic.
Reg0 is used by Classifier and LBaaS is moved to Reg1 + Reg2.
Change-Id: I619247349631f909dcc77177bd4cfe733afe7d50
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Madhu Venugopal [Mon, 22 Sep 2014 06:13:31 +0000 (06:13 +0000)]
Merge "Fixing files in resources that are for debugging help."
Madhu Venugopal [Sun, 21 Sep 2014 17:12:33 +0000 (10:12 -0700)]
Bug 2008 - Local Instance rules are failed to get programmed in L2FwdService table in presence of security groups
Change-Id: I0f628ba0099e3acfbbf06dfb381d35317647970d
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Srini Seetharaman [Sun, 21 Sep 2014 15:08:46 +0000 (08:08 -0700)]
Fixing files in resources that are for debugging help.
1. Fixing puppet bootstrap for vagrant initialization. Currently,
it references to puppetlabs-release-saucy.deb that doesn't exist.
2. Adding postman collection for the LBaaS
Change-Id: I915d9a73e9865fb16d910b0971907acee63fc65c
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Flavio Fernandes [Fri, 19 Sep 2014 18:59:40 +0000 (14:59 -0400)]
Bug 1981: NumberFormatException in ArpResponderService while programming Static ARP entries
The function ActionUtils.nxLoadArpShaAction() can now take a BitInteger or MacAddress.
Change-Id: Ia53a1e32d33b69f2ec0453d5c816d5f8c404814d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Madhu Venugopal [Sun, 21 Sep 2014 04:45:24 +0000 (04:45 +0000)]
Merge "Add missing Order and Key to InstructionBuilders"
Madhu Venugopal [Sun, 21 Sep 2014 04:11:56 +0000 (04:11 +0000)]
Merge "Bug 1885: VM ingress rule is not installed for vlan networking"
Madhu Venugopal [Sun, 21 Sep 2014 04:05:30 +0000 (04:05 +0000)]
Merge "Fix ARP Responder Actions"
Brent Salisbury [Sat, 20 Sep 2014 19:12:40 +0000 (19:12 +0000)]
Merge "Bug 1998 - Fixing Bridge Set-Controller operation failure"
Srini Seetharaman [Sat, 20 Sep 2014 17:14:44 +0000 (10:14 -0700)]
Bug 2001 - Removing code that deletes flow state when a node is removed.
This patch ensures the TransactionCommitFailedException is not being caused.
We let the MD-SAL deal with clearing state and bringing it back when the node
is removed and added back.
Change-Id: Id743b9bfb83e4424f239e05dc0e731fc818f6468
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Madhu Venugopal [Sat, 20 Sep 2014 12:36:17 +0000 (05:36 -0700)]
Bug 1998 - Fixing Bridge Set-Controller operation failure
If Controller-Target already exist in the Controller table, the existing logic in setOFController
fails badly due to a very basic bug of updating a wrong table.
If the Target exist in the Controller table, the updateRow must happen only on the Bridge Table
and Not on Controller Table. That is the root-cause of the issue seen recently.
While reviewing this bug, I felt that the logic sorrounding this is unncessarily complex and I took
this opportunity to clean it a bit as well.
Change-Id: I14cd38759c16f8b444d2d14bd0b83d1df5a3b45d
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Dave Tucker [Fri, 19 Sep 2014 15:36:36 +0000 (16:36 +0100)]
Add missing Order and Key to InstructionBuilders
Fixes bug 1982
Change-Id: Iea2a4ed1db2d9099e2dd81fc99685db505cdc521
Signed-off-by: Dave Tucker <djt@redhat.com>
Madhu Venugopal [Fri, 19 Sep 2014 14:11:36 +0000 (07:11 -0700)]
Bug 1979 - Karaf : odl-adsal-compatibility feature needs to be pulled in automatically for odl-adsal-openstack
Change-Id: Ic395e8b4dd36cf96ebcd81daa8441ef6295a06c8
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Dave Tucker [Fri, 19 Sep 2014 14:06:03 +0000 (15:06 +0100)]
Fix ARP Responder Actions
Fixes bug 1980
Change-Id: Iac673f7022b4059399b93e21ed4d1ce73d4ff99d
Signed-off-by: Dave Tucker <djt@redhat.com>
Madhu Venugopal [Fri, 19 Sep 2014 05:50:27 +0000 (22:50 -0700)]
Bug 1972 - Flows failed to get programmed in a very random fashion.
Fixing it by introducing a little bit of a delay in writeFlow. This solves the pipeline flow programming woes.
Also eventdispatcher quits if there is atleast 1 exception. This can result in event queue backing up completely.
Change-Id: Ifc8a5d16e4ae332281b6f2ea8e99788dc8c42a49
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Srini Seetharaman [Tue, 16 Sep 2014 21:16:36 +0000 (14:16 -0700)]
Bug 1897 - Implementing NeutronLoadBalancerPoolAware interface
1. Adding LBaaSPoolHandler class and changes to Activator
2. Allowing any order of VIP and Pool creation, instead of Pool
and members first as it was implemented earlier.
3. Removing hack in LBaaSPoolMemberHandler member deletion and
recomputing indices inline without enqueuing another event.
Note: With current implementation:
- It is possible to have more than 1 VIP per pool
- Create pool and VIP in any order
- There can only be 1 VIP per subnet.
- Two pools cannot share the same VIP.
Change-Id: I38d2adf07b70393a51d5ceaa467022426f3146fc
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Srini Seetharaman [Tue, 16 Sep 2014 17:07:08 +0000 (10:07 -0700)]
Fixing Bug 1900 and 1908
Updating LBaaSPoolMemberHandler to process member add/delete correctly.
Previously the rules were not pushed or deletede properly.
Also committing the index subtraction in LoadBalancerService that was
not merged properly in the earlier commit.
Change-Id: If367895407a284333a0438ae76312063fe3d8ded
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Srini Seetharaman [Mon, 15 Sep 2014 20:36:02 +0000 (13:36 -0700)]
Fixing Bug 1858 and Bug 1673 with changes in net-virt package
1. Adding support for pool-id based on fix that went into the controller/
northbound/networkconfig/neutron for fixing Bug 1673
2. Fixing issue where LB pool member creation / deletion fails when VIP
isn't created yet
3. Incorrect handling of the events (missing "break" in switch statement)
4. Adding missing Activator .setInterface for LBaaS handlers
Change-Id: Ied854fe4f44c839a1e46036e625b7fb3cf3111cc
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Srini Seetharaman [Mon, 15 Sep 2014 20:11:00 +0000 (13:11 -0700)]
Bug 1890 - Fixing NX-Multipath action plugin
1. Setting right type for Dst as DstChoice
2. Updating convertor class to do right translation between OFJava and SAL
3. Updating ActionUtil call that uses the NX-Multipath action
Change-Id: Id226bab612cd3fa0c1243a00955e5a187fd7706a
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>
Dave Tucker [Tue, 16 Sep 2014 19:19:07 +0000 (19:19 +0000)]
Merge "Bug 1896: L3 router interface not being installed"
Ed Warnicke [Tue, 16 Sep 2014 16:55:25 +0000 (11:55 -0500)]
Bug 1909: fix maven-bundle-plugin version
Change-Id: I962960f9180065080d3c32a3a14614eb0bbea452
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Flavio Fernandes [Tue, 16 Sep 2014 00:00:15 +0000 (20:00 -0400)]
Bug 1896: L3 router interface not being installed
Fix code path where tenant instance is added to node after router interface
is created.
Patch 2: code review. Thanks Mr. T! :)
Change-Id: I466f1c45cde8f3002b20cfb0f43b9e791a0031bf
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Madhu Venugopal [Mon, 15 Sep 2014 18:32:50 +0000 (11:32 -0700)]
Bug 1889 - Adding the missing odl-openflowplugin-flow-services feature to odl-ovsdb-openstack feature
Change-Id: I4ab2ff86deae6c82a0d39f28169e1f31fddbc9b0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Madhu Venugopal [Mon, 15 Sep 2014 17:44:15 +0000 (10:44 -0700)]
Reording bundle dependencies in features.xml to solve Bug 1887
Change-Id: I475b9c486056d4eb8b80a8fd3be6406ad9f868e6
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Sam Hague [Mon, 15 Sep 2014 17:12:30 +0000 (13:12 -0400)]
Bug 1885: VM ingress rule is not installed for vlan networking
Related to Bug 1421 in openflowplugin where changes were made to how the vlan related match and actions are implemented. The current openflowplugin requires that the vlan match with present = false be added as a match before the mod_vlan_vid action will work. The patch adds the match condition.
There was an existing method to add the vlan vid and present flag. The method was enahanced to allow setting the present flag to true or false.
Change-Id: Ib69149649a946098ffd988de14663b71e958a551
Signed-off-by: Sam Hague <shague@redhat.com>
Madhu Venugopal [Sun, 14 Sep 2014 21:07:02 +0000 (14:07 -0700)]
Bug 1825 - L2Fwding service should not inherit actions from the AbstractServiceInstance
Due to the Write-Actions vs Apply-Actions issue, L2Fwd service is moved to the end of the pipeline.
When it is moved, it should not inherit the Drop-flow from AbstractServiceInstance parent.
Change-Id: I14acc18ca0e9a9485d2cfcea53a4a0910f9db35e
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Madhu Venugopal [Sun, 14 Sep 2014 19:02:48 +0000 (12:02 -0700)]
Bug 1864 - Use unique flow-id for AbstractService pipeline flows
Change-Id: I537436bdaad7bbea44ebe49b999d2dfd45f71ce5
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Dave Tucker [Fri, 12 Sep 2014 23:47:30 +0000 (00:47 +0100)]
Create 3 new bundles for config files
Fixes bug 1823
Change-Id: I24d206838e07c3b8f5529d9daecc0e6112f0bfc8
Signed-off-by: Dave Tucker <djt@redhat.com>
Dave Tucker [Thu, 11 Sep 2014 23:51:31 +0000 (00:51 +0100)]
Add config file for nicira-extension to karaf distribution
Partially fixes bug 1823
Change-Id: I55ac45e2c3d4c6fa187260d839d3824f513c5705
Signed-off-by: Dave Tucker <djt@redhat.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Flavio Fernandes [Fri, 12 Sep 2014 12:38:52 +0000 (08:38 -0400)]
Bug 1844 : Callback from neutron's router interface to be handled via neutron port events
In the absence of explicit router interface add/remove callback, check the interface
owner of the neutron port and determine if that represents a router interface. From there,
simply map the event as the expected router interface event.
Change-Id: Ib5a58ad166e632b89cc97a23b2c8545f6462ca66
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Madhu Venugopal [Fri, 12 Sep 2014 12:49:45 +0000 (12:49 +0000)]
Merge "Bug 1823 : Added karaf bundle for 43-openflowjava-nx-api-config.xml"
Madhu Venugopal [Fri, 12 Sep 2014 12:49:31 +0000 (12:49 +0000)]
Merge "Bug 1823 : Adding Karaf openflowjava-nx-config bundle"
Madhu Venugopal [Fri, 12 Sep 2014 08:44:01 +0000 (01:44 -0700)]
Bug 1823 : Added karaf bundle for 43-openflowjava-nx-api-config.xml
Change-Id: I982a281640112f5e6ac023a0763024664189b66c
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
Brent Salisbury [Thu, 11 Sep 2014 23:56:51 +0000 (19:56 -0400)]
Bug 1823 : Adding Karaf openflowjava-nx-config bundle
Patch#4: Fixed a few more missing items that makes it integration-friendly
Patch#3: forgot I had dskiptests aliased from long build times.
This builds :0
Patch2: Deleting a config file a config file from before
we realized it needed to be new bundles.
Change-Id: I9cb6fe69eadc8df9bcffebfc3e15639e491db662
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>
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>
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>
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>
Dave Tucker [Thu, 11 Sep 2014 16:36:20 +0000 (16:36 +0000)]
Merge "Bug 1799: Fixed broken Vagrantfile URL"
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."
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>
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>
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>
Dave Tucker [Tue, 9 Sep 2014 12:43:33 +0000 (12:43 +0000)]
Merge "Bug 1760 - Removing redudant call to writeFlow in L3ForwardingService"
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>
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>
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>
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>
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"
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>
Brent Salisbury [Fri, 5 Sep 2014 14:57:08 +0000 (14:57 +0000)]
Merge "Adding SFC feature dependencies to ovs-sfc module"
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>
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"
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"
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>
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>
Madhu Venugopal [Fri, 5 Sep 2014 09:48:22 +0000 (09:48 +0000)]
Merge "Combine ovssfc patches."
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>
Brent Salisbury [Thu, 4 Sep 2014 21:45:40 +0000 (21:45 +0000)]
Merge "Added LoadBalancerProvider as the service provided by LoadBalancerService"
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"
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."
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>
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>
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>
Dave Tucker [Thu, 4 Sep 2014 12:53:20 +0000 (12:53 +0000)]
Merge "L3: Add support for distributed arp responder and l3 forwarding"
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>
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>
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>
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>
Madhu Venugopal [Thu, 4 Sep 2014 01:31:31 +0000 (01:31 +0000)]
Merge "ACL services for OS/OVS"
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>
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>
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>
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>
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>
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>
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>
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>
Madhu Venugopal [Wed, 3 Sep 2014 11:01:30 +0000 (11:01 +0000)]
Merge "Updating the LB pool member DELETE and UPDATE processing."
Madhu Venugopal [Wed, 3 Sep 2014 11:01:18 +0000 (11:01 +0000)]
Merge "Adding support for programming newly added nodes."
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."
Madhu Venugopal [Wed, 3 Sep 2014 10:48:31 +0000 (10:48 +0000)]
Merge "Net-Virt LBaaS module updates for neutron event handling"
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."
Dave Tucker [Wed, 3 Sep 2014 09:55:20 +0000 (09:55 +0000)]
Merge "L3: Break exclusions and rewrites into inbound and outbound conterparts."
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>
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>
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>
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>
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>
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>
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."
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)"
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>
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>
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>
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>
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>
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>