Anil Vishnoi [Thu, 7 May 2015 10:50:54 +0000 (16:20 +0530)]
Fixed OvsdbDataChangeListner bridge update processing
Change-Id: Ie3414edce90795ce72823bbd8e93b0b4ce605c03
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sharad Mishra [Wed, 6 May 2015 18:17:53 +0000 (11:17 -0700)]
Implemented processing of Bridge Update
Change-Id: I5d3c212950a37df81fc717b77f0b36fb826a969a
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sam Hague [Thu, 7 May 2015 01:18:39 +0000 (21:18 -0400)]
Keep netvirt pom exporting packages
Change-Id: Ie54bf4a0d7a09bacbceecf9b48d5276b22c582ac
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 7 May 2015 00:55:25 +0000 (20:55 -0400)]
Merge branch 'master' into topic/netvirtsb_merge
Conflicts:
openstack/net-virt/pom.xml
openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/api/NetworkingProviderManager.java
openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/api/TenantNetworkManager.java
Change-Id: I26a8d6fba1f59504bd16da60168a420118d6c558
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 7 May 2015 00:29:22 +0000 (00:29 +0000)]
Merge "Southbound-IT: add test to verify dpdk enabled switch."
Anil Vishnoi [Wed, 6 May 2015 23:00:24 +0000 (04:30 +0530)]
Quick and Dirty fix for bridge controller ip address
Rather the using the hardcoded value, using the local host ip address.
This is dirty fix, we should remove it once we get the valid fix using
the connectionInfo local ip address. I think local ip address issues is now
fixed for conncetion-info, so we can probably use that. We need to merge the
master to netvirtsb branch, that will bring that fix.
With this fix, netvirt create br-int and install all the pipeline flow
successfully.
Change-Id: If9d4e1289e0bb8c2a2888801f3c219f01f9a962d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 20:12:56 +0000 (01:42 +0530)]
OvsdbDataChangeListner : Implemented processing of openvswitch (ovsdb connection) updates
Change-Id: Iea1b2a4ec804afce8801ec916a08b4bc5bbe3632
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Wed, 6 May 2015 18:52:08 +0000 (14:52 -0400)]
Workaround for INeutron bundles not letting services start
Change-Id: I204c3a39f0843180915a02dcbd3287c61aa1eafc
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Wed, 6 May 2015 18:28:42 +0000 (23:58 +0530)]
OvsdbDataChangeListner : Implemented processing of port/interface update
Change-Id: Ic61f5f2c0b426b988882f9bef7553143e9c537e7
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 17:01:26 +0000 (22:31 +0530)]
OvsdbDataChangeListner : Implemented processing of bridge deletion
Change-Id: If94b5fcd6e2af2c564637d7740f48f18733686fd
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 16:21:49 +0000 (21:51 +0530)]
OvsdbDataChangeListener : Implemented processing of new bridge creation
Change-Id: I8053f7be5b89c8a4d409ba245e509fac68d783b3
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Ryan Goulding [Wed, 6 May 2015 16:01:51 +0000 (12:01 -0400)]
Fix SouthboundIT by disconnecting after each test
SouthboundIT requires disconnection from OVSDB after each test.
Change-Id: Ifb916f103e94d53049f684a64b87c0ed1fc2efd6
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 13:37:49 +0000 (19:07 +0530)]
OvsdbDataChangeListener : Implemented ovsdb node disconnection processing
Change-Id: I7edcfb0e8dd6528d6a1aaf870f71b38eca542b8c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 13:12:03 +0000 (18:42 +0530)]
OvsdbDataChangeListener : Implemented port (termination point) delete event processing
Change-Id: I6cd3ef89927aaae829815b2a9c625b6f89907162
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Ryan Goulding [Tue, 5 May 2015 15:22:57 +0000 (11:22 -0400)]
Southbound-IT: add test to verify dpdk enabled switch.
This test will
1. Check if it is a DPDK node.
2. Test exits if its not a DPDK node. If DPDK node then,
3. Create a netdev bridge.
4. Verifies that a netdev bridge was created.
5. Creates a dpdk port.
6. Verifies that a dpdk port was added.
7. deletes bridge and disconnects node.
8. Test fails if any of the 3-7 step fails.
Change-Id: I58e10ba18ed82193782b981ed35b22ee0e14fd62
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Anil Vishnoi [Wed, 6 May 2015 12:16:15 +0000 (17:46 +0530)]
ovsdbDataChangeListener : Implemented Termination point creation processing
Patch 2: rebased patch
Change-Id: I03c440609b838177f990a471ed609467d7e581be
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Wed, 6 May 2015 11:12:49 +0000 (07:12 -0400)]
Remove OvsdbConfigurationService
Change-Id: Ie33affff9de47120bc0d5d134edcdbcf96a2c5ee
Signed-off-by: Sam Hague <shague@redhat.com>
Robert Varga [Tue, 5 May 2015 14:58:01 +0000 (16:58 +0200)]
Fix javadoc JDK8 compatibility
JDK8 is more picky about javadoc correctness -- fix it up so we can pass
compilation.
Change-Id: Iae446e890afe91c6ca0b670c2ea53f18793b5e99
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Tue, 5 May 2015 14:57:26 +0000 (16:57 +0200)]
Sort reported nodes
The order of the nodes is asserted by the unit tests -- make sure this
is predictable across JDK versions.
Change-Id: I804024fe66530aa380dbd48907268fc12a4d3085
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Wed, 6 May 2015 09:05:40 +0000 (11:05 +0200)]
Remove explicit export-package
net-virt-providers/OF13Provider requires the
org.opendaylight.ovsdb.openstack.netvirt package to be visible. Instead
of hard-coding the exported packages, allow the bundle plugin to decide
what to export.
Change-Id: Iac66eeaf70854712b9dc5917c748d519178968e5
Signed-off-by: Robert Varga <rovarga@cisco.com>
Flavio Fernandes [Wed, 6 May 2015 00:37:58 +0000 (00:37 +0000)]
Merge "User provided bridge external-ids not making to operational/OVSDB."
Sam Hague [Tue, 5 May 2015 12:51:25 +0000 (08:51 -0400)]
Add missing handleInterfaceDelete for tunnel case
Change-Id: I9259fc0d377724c2e7c9959a547b255f37439960
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 12:47:56 +0000 (08:47 -0400)]
SouthboundHandler@processOpenVSwitchUpdate
Change-Id: I23c622889236310850c6b0dd5d944479a21b7574
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 12:38:07 +0000 (08:38 -0400)]
Remove old interface delete row processing
Change-Id: I2dccab918622e4c4877c4106813bdf78010f0014
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 12:26:22 +0000 (08:26 -0400)]
update NetvirtIT to include remove of flowCapableNode processing
Change-Id: Idbcb4117af744d57a014866d0d10325a13e9d91b
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 12:24:03 +0000 (08:24 -0400)]
SouthboundHandler#triggerUpdates
Change-Id: Ia2455649badbec3814ad384be743fadfd8fb071d
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 12:07:03 +0000 (08:07 -0400)]
Remove FlowCapableNodeDataChangeListener
Change-Id: I737cef9874c5ac15a2805d2e4b08aba5b371c1d6
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 5 May 2015 11:59:11 +0000 (07:59 -0400)]
Add SouthboundHandler#processBridgeUpdate
Change-Id: I1a9161fa03f9cd8778973879e40aac6bf3283a70
Signed-off-by: Sam Hague <shague@redhat.com>
Sharad Mishra [Tue, 5 May 2015 07:20:50 +0000 (00:20 -0700)]
User provided bridge external-ids not making to operational/OVSDB.
When setting up a new bridge with user provided external-ids to
config datastore, these external-ids get dropped and only the iid
shows up as bridge external-id in ovsdb/oper datastore. This patch
fixes that issue.
Change-Id: I04676e166cab96323aff0ce5747a950c111da99c
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Amit Mandke [Tue, 5 May 2015 01:15:33 +0000 (18:15 -0700)]
Fixing issue with interface update.
Currently by mistake it updates the port uuid (in operation tp) using interface uuid.
Change-Id: Ia4f3dfdb676c377dab83dda11148222640e6bf78
Signed-off-by: Amit Mandke <ammandke@cisco.com>
Sam Hague [Tue, 5 May 2015 00:57:27 +0000 (00:57 +0000)]
Merge "Fix for bridge other config in mdsal."
Sam Hague [Tue, 5 May 2015 00:53:47 +0000 (20:53 -0400)]
Add SouthboundHandler#processInterfaceUpdate
Change-Id: I413d2846b5b62d08a082c7540f150e28468e34ab
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Mon, 4 May 2015 20:58:18 +0000 (02:28 +0530)]
Implemented SouthboundHandler#processInterfaceDelete()
Change-Id: I62654de72ecc45eefba549125c6f6c6c02accd70
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Mon, 4 May 2015 20:43:01 +0000 (16:43 -0400)]
Add MdsalUtils API stubs
Change-Id: I45c91dd577732cdc84161114fd154d4f6fef5871
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 4 May 2015 19:53:32 +0000 (19:53 +0000)]
Merge "Add ovsdbUpdate() to OvsdbInventoryListener" into topic/netvirtsb
Sam Hague [Mon, 4 May 2015 19:52:21 +0000 (15:52 -0400)]
Add ovsdbUpdate() to OvsdbInventoryListener
Change-Id: Iefa15fdf20f27b95481067d5d915fc2ccf845487
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Mon, 4 May 2015 18:19:59 +0000 (23:49 +0530)]
Fixed check-style error
Change-Id: I8f70f91e3ebdd09a20740aa349d65f3ec2ab7a46
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sharad Mishra [Mon, 4 May 2015 09:47:46 +0000 (02:47 -0700)]
Fix for bridge other config in mdsal.
bridge other config are not showing up in operational or ovsdb.
Change-Id: Id1fd2f4f57fdd3aef1a25bf08229df26f6d3c76d
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sam Hague [Fri, 1 May 2015 23:50:50 +0000 (19:50 -0400)]
Get correct dpid so that pipeline flows are programmed.
This commit also fixes the initialzeOpenflowRules.
Change-Id: I838db8f727b0057066afcce1e242e1aab80ed1ac
Signed-off-by: Sam Hague <shague@redhat.com>
Flavio Fernandes [Fri, 1 May 2015 19:48:51 +0000 (15:48 -0400)]
AbstractServices need unique flowIds in order to avoid rules coalescing
This is a follow up from gerrit https://git.opendaylight.org/gerrit/#/c/19410/.
When assembling different flow rules for different match, it is important
to make their flowIds unique. By not doing that, openflowPlugin will treat
them as one; and that is what gets pushed to OF capable node.
Also removed cases when writeFlow() was being invoked twice for no reason.
Change-Id: I898d777e779e61da3c9f070306915c4c37f65572
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Ed Warnicke [Fri, 1 May 2015 17:58:52 +0000 (10:58 -0700)]
Fixed issue with connection-info->{local-ip,local-port} not being stored
Use of ConnectionInfo as a key in OvsdbConnectionManager
should only have remote{Ip,port} info... so:
1) Introduced a method suppressLocalIpPort() to SouthboundMapper
to strip localIp/Port info out of a ConnectionInfo if needed
2) Insured that *all* gets to the clients map in OvsdbConnectionManager
use that method before doing a lookup
3) Insured that *all* puts to the clients map in OvsdbConnectionManager
use that method before writing.
Also in passing fixed disconnect to just get the connectionInfo from
the ovsdbNode it has, instead of doing a convoluted dance and
removed the then no longer used createConnectionInfo(ovsdbNode) from
SouthboundMapper.
Change-Id: Idb2b16d52ec23267584c69c6e474652dd25a4ab0
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
Sam Hague [Fri, 1 May 2015 01:30:18 +0000 (21:30 -0400)]
Add support for openflow node callbacks
Change-Id: I79a8f9d1cf2ee46b240ff0ffbe467f35558f6e41
Signed-off-by: Sam Hague <shague@redhat.com>
Flavio Fernandes [Thu, 30 Apr 2015 20:17:58 +0000 (16:17 -0400)]
Work around bug 3005: RoutingService needs special care to keep from rule coalescing
When rules with for RoutingService only differ on match for tun_id, the existing openflow implementation does
not distinguish them as separate rules and overrides them into a single OF entry. That is very bad for L3
forwarding, when there are more than 2 subnets, or when multiple tenants use overlapping prefixes.
Until bug 3005 is fixed, we will leverage cookie to force openflow to treat the rule as unique. Fortunately
this is easily done by using the src_tun_id itself as the cookie (i.e. no cache is needed).
Also, add segmentation info in logs for when router interface is being programmed.
vagrant@devstack-control:/vagrant/puppet/scripts$ sudo ovs-ofctl -O OpenFlow13 dump-flows br-int | grep table=60
cookie=0x0, duration=561.894s, table=60, n_packets=219, n_bytes=26397, priority=0 actions=goto_table:70
cookie=0x3ea, duration=468.025s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3ea,nw_dst=3.0.0.0/24 actions=set_field:fa:16:3e:d6:0c:aa->eth_src,dec_ttl,set_field:0x3eb->tun_id,goto_table:70
cookie=0x3e9, duration=470.048s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3e9,nw_dst=3.0.0.0/24 actions=set_field:fa:16:3e:d6:0c:aa->eth_src,dec_ttl,set_field:0x3eb->tun_id,goto_table:70
cookie=0x3eb, duration=467.009s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3eb,nw_dst=2.0.0.0/24 actions=set_field:fa:16:3e:2d:28:89->eth_src,dec_ttl,set_field:0x3ea->tun_id,goto_table:70
cookie=0x3eb, duration=469.035s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3eb,nw_dst=1.0.0.0/24 actions=set_field:fa:16:3e:25:0d:1b->eth_src,dec_ttl,set_field:0x3e9->tun_id,goto_table:70
cookie=0x3e9, duration=489.275s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3e9,nw_dst=2.0.0.0/24 actions=set_field:fa:16:3e:2d:28:89->eth_src,dec_ttl,set_field:0x3ea->tun_id,goto_table:70
cookie=0x3ea, duration=488.184s, table=60, n_packets=0, n_bytes=0, priority=2048,ip,tun_id=0x3ea,nw_dst=1.0.0.0/24 actions=set_field:fa:16:3e:25:0d:1b->eth_src,dec_ttl,set_field:0x3e9->tun_id,goto_table:70
Ref: https://gist.github.com/
f3ba1edbab72b26c8d97
Ref: http://www.flaviof.com/blog/work/how-to-odl-with-openstack-part2.html
Change-Id: I7eb43b141f26b09a1c5870a7d494bc9c2cebecd8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Sam Hague [Thu, 30 Apr 2015 16:30:01 +0000 (12:30 -0400)]
Merge branch 'master' into topic/netvirtsbmerge
Change-Id: I1da78555e1e5a29f9b195ceedf508fc597a025a5
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 30 Apr 2015 16:27:07 +0000 (16:27 +0000)]
Merge "Add testGetOvsdbNodes to it"
Sam Hague [Thu, 30 Apr 2015 16:25:22 +0000 (12:25 -0400)]
Set the controller
Change-Id: Ib62eaee4a5a95032f5a4b4197d4fe24e861d012c
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 30 Apr 2015 16:11:51 +0000 (12:11 -0400)]
Add testGetOvsdbNodes to it
Change-Id: I33d1051bad7a220d14e569642a358ed9d929acf0
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 30 Apr 2015 11:57:27 +0000 (07:57 -0400)]
Add is_connected to mdsal
Change-Id: I6ddfb22afbb2b4a27887602a8e00fa15a676d217
Signed-off-by: Sam Hague <shague@redhat.com>
Thomas Bachman [Wed, 29 Apr 2015 19:57:33 +0000 (15:57 -0400)]
Fix for NPE during GBP testing.
This addresses an NPE that happens if the
controller entry "isConnected" isn't set but
is referenced.
Signed-off-by: Thomas Bachman <tbachman@yahoo.com>
Sam Hague [Wed, 29 Apr 2015 00:43:27 +0000 (20:43 -0400)]
Merge branch 'master' into topic/netvirtsb2
Change-Id: I70ee2e158f4ed2b97c51861e4b44aa2306740694
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Wed, 29 Apr 2015 00:39:06 +0000 (20:39 -0400)]
Process ovsdb node created dataChanges
Change-Id: I084cc1e3cf1cc3b499cef858cc9a8ae0fac32466
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 28 Apr 2015 19:35:30 +0000 (19:35 +0000)]
Merge "Add OpenFlow node reference to the bridge."
Sharad Mishra [Tue, 28 Apr 2015 08:45:21 +0000 (01:45 -0700)]
Add bridge fails on older OVS versions (< v1.10)
To recreate -
Run "ovs-vsctl add-br br0" on OVS 1.9 or lower.
The above command fails with SchemaVersionMismatchException.
Change-Id: Iee3adda5252e50e17c493373f74ee26e928f524f
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sam Hague [Tue, 28 Apr 2015 11:20:07 +0000 (11:20 +0000)]
Merge "Update TerminationPoint operational data on ovsdb port/interface update"
Anil Vishnoi [Sat, 25 Apr 2015 00:36:30 +0000 (06:06 +0530)]
Update TerminationPoint operational data on ovsdb port/interface update
Change-Id: Id6f78e233c4c165d3f7e49a173794d08510abb65
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sharad Mishra [Mon, 27 Apr 2015 19:00:41 +0000 (12:00 -0700)]
Fix NPE in operational Bridge remove.
To Recreate -
1. Run 'ovs-vsctl add-br br0'
2. Verify that bridge is added.
3. Run 'ovs-vsctl del-br br0'
4. The logs show NPE and br0 is not removed from operational/ovsdb.
Change-Id: I742a0661e97521047ee1c16c2b0a45e3a2a1f1cd
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sam Hague [Mon, 27 Apr 2015 14:54:37 +0000 (10:54 -0400)]
Merge branch 'master' into topic/netvirtsb2
Signed-off-by: Sam Hague <shague@redhat.com>
Conflicts:
southbound/southbound-it/src/test/java/org/opendaylight/ovsdb/southbound/it/AbstractConfigTestBase.java
southbound/southbound-it/src/test/java/org/opendaylight/ovsdb/southbound/it/SouthboundIT.java
Change-Id: I226c8118b399e60ef846d165a701f62873324739
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 27 Apr 2015 14:49:55 +0000 (10:49 -0400)]
Add working integration-test
Change-Id: I7fb4cdb4b897bad022055ff14a4e2d60a6f7fa78
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 27 Apr 2015 11:20:25 +0000 (07:20 -0400)]
Remove assumes from southbound-it now that deletes from operational work
Change-Id: I8493086453f09b8aac6d4825543bce2d00191ec1
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sun, 26 Apr 2015 19:27:25 +0000 (15:27 -0400)]
Merge branch 'master' into topic/netvirtsb2
Signed-off-by: Sam Hague <shague@redhat.com>
Conflicts:
openstack/net-virt/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/NetworkHandlerTest.java
Signed-off-by: Sam Hague <shague@redhat.com>
Change-Id: I75654ecea55d84cd022b856e2b0a30a1144a6f05
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sun, 26 Apr 2015 18:33:48 +0000 (14:33 -0400)]
More southbound migration for netvirt
Change-Id: Ie972ab5914cc2a209fb73b53936c76269590f75a
Signed-off-by: Sam Hague <shague@redhat.com>
Alexis de Talhouët [Thu, 23 Apr 2015 20:24:07 +0000 (16:24 -0400)]
Add OpenFlow node reference to the bridge.
Change-Id: Id000733a51f35841c5ddbd14b57887418e30f16d
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Anil Vishnoi [Fri, 24 Apr 2015 13:08:02 +0000 (18:38 +0530)]
Fixed termination point deletion from operational data store.
Plugin was not removing termination point operational data from
operational data store once port deleted from bridge
Change-Id: I61de3e9b5dcdc4ac096422f6445562a194ba9fa1
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Sat, 18 Apr 2015 21:30:48 +0000 (14:30 -0700)]
Added detailed description to external-ids and other-configs in ovsdb.yang
Detailed descriptions from OpenVswitch schema were added to external-ids
and other-configs in ovsdb.yang file.
But, the javadoc created as a result of new description is not correctly
formatted. Any help here will be appreciated.
Patch 2: Fixed whitespaces
Change-Id: I0016fd02a76829fe4aa20a6967eb21c5a379eb06
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Fri, 24 Apr 2015 02:05:52 +0000 (02:05 +0000)]
Merge "Fix for exception thrown when no ovs_version set."
Sam Hague [Fri, 24 Apr 2015 01:33:49 +0000 (01:33 +0000)]
Merge "SouthboundIT: Test to check ovs version for node."
Cédric Ollivier [Thu, 23 Apr 2015 17:03:40 +0000 (19:03 +0200)]
Delete redundant tests in Network, Port and Subnet handlers
This removes redundant tests according to Neutron*Northbound.java.
HTTP_OK is returned now if the neutron resource can be created, updated
or deleted.
Change-Id: I621121f020123653da8ef87fef62a27ab951629e
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>
Cédric Ollivier [Tue, 3 Feb 2015 09:14:48 +0000 (10:14 +0100)]
Prevent the network shared attribute from being updated
This modifies canUpdateNetwork() to check the network shared
attribute.
Change-Id: Ie875c4cc437c27f6b202e4e26dc1acd1aeff41f8
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>
Sharad Mishra [Thu, 23 Apr 2015 16:52:38 +0000 (09:52 -0700)]
Fix for exception thrown when no ovs_version set.
I am running a DPDK accelerated OVS that was manually built.
It does not have its ovs_version field set and ODL throws exception.
Not sure if we support such ovs ... is it a support statement issue or bug ??
Change-Id: I778c89e241120058b380c96b2b074eab255a85bd
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sharad Mishra [Thu, 23 Apr 2015 15:16:37 +0000 (08:16 -0700)]
SouthboundIT: Test to check ovs version for node.
Starting small ... this patch checks ovs version attribute
of ovs node.
I could have added this check to "testAddDeleteOvsdbNode', but decided
that it could lead to minor confusion.
Change-Id: Iab4bf14215a2ced55f3c05be5f5a083db190f286
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Ryan Goulding [Wed, 22 Apr 2015 15:03:58 +0000 (11:03 -0400)]
SouthboundIT for vlan, vlanmode and vlan trunks
Add integration tests to verify vlan, vlan mode and vlan trunks functionality.
Change-Id: I745ba976a244b42d7f38a9d43db8eacac77ba416
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Sam Hague [Tue, 21 Apr 2015 19:54:54 +0000 (15:54 -0400)]
Merge branch 'master' into topic/netvirtsb2
Change-Id: I8555d101f09b2e347a0ecf199dbbb3ce3be81905
Signed-off-by: Sam Hague <shague@redhat.com>
Flavio Fernandes [Tue, 21 Apr 2015 19:51:11 +0000 (19:51 +0000)]
Merge "additional southbound-it for port/interface"
Ryan Goulding [Tue, 21 Apr 2015 15:43:34 +0000 (11:43 -0400)]
additional southbound-it for port/interface
Adds integration tests to verify ofPortRequest, termination point options,
interface external-ids, port external-ids, interface other-configs, and port
other-configs.
Change-Id: Ibf76a20a5844885e404b53d4a9c163452ca4084a
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Sam Hague [Tue, 21 Apr 2015 14:54:40 +0000 (10:54 -0400)]
Update poms for Maven Central Requirements
Change-Id: I73fa6324ea917594e5cd110acd8f7b68102b3ca1
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 21 Apr 2015 02:52:54 +0000 (22:52 -0400)]
Use Topology Node in place of Inventory Node
Change-Id: I8aa774acc6d617036d5803716619573df52e0697
Signed-off-by: Sam Hague <shague@redhat.com>
Ryan Goulding [Mon, 20 Apr 2015 21:14:57 +0000 (17:14 -0400)]
ofPort test in SouthboundIT
Adds an integration test to verify ofPort functionality. A helper method is
provided to create a new TerminationPoint for testing.
Change-Id: I4588b9c95dff067007ebec47ea268c0f4e1f1bdf
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Sam Hague [Mon, 20 Apr 2015 19:07:49 +0000 (15:07 -0400)]
Replace plugin interfaces with net-virt interfaces.
This commit simply defines the interfaces that were removed in the previous commit. The next step from here is to modify the interfaces to be more generic so if the underlying southbound changes the interface does not need to change.
Change-Id: I6778b1019e05e3eb7e5d2e48693ca980f17bebdd
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 20 Apr 2015 15:46:48 +0000 (11:46 -0400)]
Remove plugin dependencies
Change-Id: I9c6d6c497c8651421549b91009ab337d294d37ba
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Mon, 20 Apr 2015 15:28:20 +0000 (20:58 +0530)]
Fix ovsdb node disconnect - Removing ovsdb node from config data store
is not disconnecting the node.
Change-Id: I15012381814b80b887f4fb0cb5b62dbcbe9eb458
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Mon, 20 Apr 2015 14:41:23 +0000 (10:41 -0400)]
Merge branch 'master' into topic/netvirtsb
Change-Id: I6fdc1f6d3f80e3acfc8baf1c5e411815dcd6d7a0
Signed-off-by: Sam Hague <shague@redhat.com>
Ben Eze [Sun, 19 Apr 2015 23:47:24 +0000 (19:47 -0400)]
Add Unit tests for openstack/netvirt/providers/openflow13/OF13Provider.java
Change-Id: I7b92343cfb7653e44755bb54afcc003b1adebcb9
Signed-off-by: Ben Eze <beze@inocybe.ca>
Sam Hague [Sun, 19 Apr 2015 19:30:00 +0000 (15:30 -0400)]
Fix broken SouthboundIT test
The mdsalUtils field needs to be static becuase each @Test method is a totally new allocation and the value is reaallocated for each test.
Also added some extra debug logs.
Patchset 2: checkstyle
Change-Id: Ic75b5bc25cdf62b1c3daa4ee2fdea487804fb0a9
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sun, 19 Apr 2015 17:55:00 +0000 (17:55 +0000)]
Merge "TerminationPointUpdate command to update termination point on ovsdb node"
Prateek Garg [Fri, 17 Apr 2015 19:55:31 +0000 (12:55 -0700)]
TerminationPointUpdate command to update termination point on ovsdb node
Change-Id: I17c4bd890a120b4fbf74ff5288f97bf00fd224df
Signed-off-by: Prateek Garg <prategar@cisco.com>
Sam Hague [Sat, 18 Apr 2015 01:22:37 +0000 (01:22 +0000)]
Merge "Remove .../distribution/opendaylight-karaf/. It is now .../karaf"
Flavio Fernandes [Fri, 17 Apr 2015 21:20:46 +0000 (21:20 +0000)]
Merge "Add JUnit testing for L2RewriteService class."
Flavio Fernandes [Fri, 17 Apr 2015 21:20:35 +0000 (21:20 +0000)]
Merge "Add JUnit testing for L2FowardingService class."
Flavio Fernandes [Fri, 17 Apr 2015 21:20:28 +0000 (21:20 +0000)]
Merge "Add JUnit testing for LoadBalancerService class."
Alexis de Talhouët [Fri, 17 Apr 2015 14:40:10 +0000 (10:40 -0400)]
Add JUnit testing for IngressAclService class.
Change-Id: I44c088d640413d978cbbec9fa67c8a5d7d0aea8b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Alexis de Talhouët [Fri, 17 Apr 2015 18:46:27 +0000 (14:46 -0400)]
Add JUnit testing for LoadBalancerService class.
Change-Id: I18fb554389d4baa05e0f58a538243cdaa456cc5a
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Alexis de Talhouët [Fri, 17 Apr 2015 17:18:09 +0000 (13:18 -0400)]
Add JUnit testing for L2FowardingService class.
Change-Id: I1d7b565f1cfa81b78423827d277e8cd5929dd51a
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Alexis de Talhouët [Fri, 17 Apr 2015 19:08:38 +0000 (15:08 -0400)]
Add JUnit testing for L2RewriteService class.
Change-Id: I902ce652c3edb0857cca697ead04a9ffd475db67
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Flavio Fernandes [Fri, 17 Apr 2015 20:44:46 +0000 (16:44 -0400)]
Remove .../distribution/opendaylight-karaf/. It is now .../karaf
Change-Id: I7287ac40e9a2cfc25e3551662cde4a32bf0b88e8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Sam Hague [Fri, 17 Apr 2015 20:20:25 +0000 (20:20 +0000)]
Merge "Move ovsdb distribution to use karaf-parent"
Sam Hague [Fri, 17 Apr 2015 19:31:27 +0000 (15:31 -0400)]
Move ovsdb distribution to use karaf-parent
Change-Id: Ief6939eb8f1cc15f1042ecc1fd7e549c60de57cc
Signed-off-by: Sam Hague <shague@redhat.com>
Flavio Fernandes [Fri, 17 Apr 2015 18:43:07 +0000 (18:43 +0000)]
Merge "Add JUnit testing for OutboundNatService class."
Flavio Fernandes [Fri, 17 Apr 2015 18:42:58 +0000 (18:42 +0000)]
Merge "Add JUnit testing for L3FowardingService class."
Flavio Fernandes [Fri, 17 Apr 2015 18:42:17 +0000 (18:42 +0000)]
Merge "Add JUnit testing for EgressAclService class."
Flavio Fernandes [Fri, 17 Apr 2015 18:40:36 +0000 (18:40 +0000)]
Merge "Add JUnit testing for RoutingService class."