Sam Hague [Fri, 8 May 2015 20:32:26 +0000 (16:32 -0400)]
Add NeutronL3Adapter back now that neutron wires correctly
Change-Id: Ia9cd61d11454b34f01ea7fe7675030bee8759ef4
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 8 May 2015 20:28:28 +0000 (16:28 -0400)]
Merge branch 'master' into topic/netvirtsb_merge
Conflicts:
southbound/southbound-it/src/test/java/org/opendaylight/ovsdb/southbound/it/SouthboundIT.java
Change-Id: Ieb1fe28a53b303e576cb060ff751cf02e1762ff0
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 8 May 2015 17:26:12 +0000 (17:26 +0000)]
Merge "MdsalUtils#getOtherConfig() & MdsalUtils#getExternalIds()." into topic/netvirtsb
Sam Hague [Fri, 8 May 2015 17:24:47 +0000 (17:24 +0000)]
Merge "Fixed few things: ** Added support for deleting bridge from config data store to keep configuration sane *.* Fixed bridge creation (it was not creating integration bridge when second compute node connects" into topic/netvirtsb
Sam Hague [Fri, 8 May 2015 17:23:50 +0000 (17:23 +0000)]
Merge "Fixing log level : Too much log making debugging hard" into topic/netvirtsb
Sam Hague [Fri, 8 May 2015 17:23:24 +0000 (17:23 +0000)]
Merge "Implemented getAllBridgedOnOvsNode" into topic/netvirtsb
Alexis de Talhouët [Fri, 8 May 2015 15:15:27 +0000 (11:15 -0400)]
MdsalUtils#getOtherConfig() & MdsalUtils#getExternalIds().
Patch Set 1: useless formatting
Patch Set 2: remove trailing space
Patch Set 3&4: MdsalUtils#getOtherConfig()
Patch Set 5: remove trailing space
Patch Set 6: MdsalUtils#getExternalIds()
Change-Id: Ie5db49990f413f12052cdb44f82a0b4a0b3ea51b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Anil Vishnoi [Fri, 8 May 2015 13:03:15 +0000 (18:33 +0530)]
Fixed few things:
** Added support for deleting bridge from config data store
to keep configuration sane
*.* Fixed bridge creation (it was not creating integration bridge
when second compute node connects
Change-Id: I4233886f875b188f7d5d7e68fe8e0076b4365d82
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Fri, 8 May 2015 12:11:42 +0000 (17:41 +0530)]
Fixing log level : Too much log making debugging hard
Change-Id: Ic62bd38b888ca98befb3cff623c12a1f28544300
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sharad Mishra [Thu, 7 May 2015 14:26:34 +0000 (07:26 -0700)]
Implemented getAllBridgedOnOvsNode
Change-Id: I00fc9b1c4b8e90752eb7cd0ed1eaa31d78c3bc59
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Sharad Mishra [Thu, 7 May 2015 15:05:40 +0000 (08:05 -0700)]
Implemented addPatchTerminationPoint()
Change-Id: I709b608f2dc6776de2159677dbcb99737f1ca5e2
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Anil Vishnoi [Thu, 7 May 2015 21:29:26 +0000 (02:59 +0530)]
MdsalUtils : Delete termination point from bridge
Change-Id: If5af04f6baffbb65959d4859c56a43a6c1c2a276
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Thu, 7 May 2015 21:11:43 +0000 (17:11 -0400)]
Reenable neutron I*Aware services
Change-Id: I322a25efbff708327204c6264423f6848f4fb56f
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 7 May 2015 20:57:46 +0000 (20:57 +0000)]
Merge "Remove mdsal-node utils" into topic/netvirtsb
Anil Vishnoi [Thu, 7 May 2015 20:51:56 +0000 (02:21 +0530)]
MdsalUtils : Implemented create tunnel termination point on bridge node
Change-Id: I639a73d30f4a69d0596a8fa889c5e9d7ab4acd52
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Thu, 7 May 2015 20:21:35 +0000 (01:51 +0530)]
MdsalUtils : Add internal termination point on ovsdb bridge
Change-Id: I0cb0ef73d4fcf6740dd0d45539eb24ada08a5cd0
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Thu, 7 May 2015 18:38:28 +0000 (14:38 -0400)]
Remove mdsal-node utils
Change-Id: I234a6cc985846ebe0dc1f66debba8ea38f72e9a4
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Thu, 7 May 2015 16:24:11 +0000 (21:54 +0530)]
Refactor MdSalUtils class to
*.* Clean up the duplication methods
*.* Refactor methods to avoid confusion whether it's reading from data store
or extracting from provided method
*.* Arranged code for better readability
Change-Id: Ib1a35ce76549c437f30b5eff7bb62959a21b676e
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Thu, 7 May 2015 17:44:40 +0000 (13:44 -0400)]
Remove lib and schema from netvirt
Change-Id: I687fca4b90b7d5a78cb7879e3cf21f50c61a7129
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Thu, 7 May 2015 15:28:24 +0000 (20:58 +0530)]
MdSalUtils : Implemented getPorts from {node, operational data store}
Change-Id: I2fd2f552960c99012d743c2863690568ea0f49c1
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Ryan Goulding [Thu, 7 May 2015 11:48:45 +0000 (07:48 -0400)]
SouthboundIT tests for Bridge external_ids and other_config
Tests create, read and delete functionality for bridge external_ids and
other_config. Future changesets will utilize work here to test update
of bridges.
Change-Id: Icd45d19c58e4838415cccb06a3d4f08b117d678f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
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>
Sam Hague [Thu, 7 May 2015 00:29:22 +0000 (00:29 +0000)]
Merge "Southbound-IT: add test to verify dpdk enabled switch."
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>
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>