Merging with mainline.
Squashed commit of the following:
commit
44b0e179a279b1dd6224248eb4fbdd3a0ef1ab08
Merge:
c2ceab5 fa2a221
Author: Sam Hague <shague@redhat.com>
Date: Wed Feb 17 21:56:48 2016 +0000
Merge "Bug 5351 Connection state tracking bit values are changed to match the Ovs2.5 implementiaon."
commit
c2ceab58e68eae1c6ea78d5f7a91fba0168730b2
Merge:
503eb83 4aa59f9
Author: Sam Hague <shague@redhat.com>
Date: Wed Feb 17 21:56:12 2016 +0000
Merge "removing ovsdb-all feature"
commit
503eb83c29271112468b2357d71b495f6109dfd3
Merge:
92d8401 864c579
Author: Sam Hague <shague@redhat.com>
Date: Wed Feb 17 21:55:53 2016 +0000
Merge "Remove references to odl-ovsdb-all"
commit
864c5795006c5a1e9c4b3d73e801377340c664c4
Author: Stephen Kitt <skitt@redhat.com>
Date: Wed Feb 17 09:47:15 2016 +0100
Remove references to odl-ovsdb-all
Now that the feature has been removed, update the documentation to
avoid listing it in the examples.
Change-Id: Ibd40e2088df955b8ab46cfcde354f364b1419cb6
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
4aa59f99b92bc3e823377b7758cedab8e943978a
Author: Jamo Luhrsen <jluhrsen@redhat.com>
Date: Tue Feb 16 13:48:45 2016 -0800
removing ovsdb-all feature
- from the looks of the featuretree graph [0] it seems that
ovsdb-all is not providing any benefit and probably would
confuse an end-user, since all it would install is the
ovsdb-library, which is pulled in by other features as
needed.
[0] https://wiki.opendaylight.org/view/OVSDB_Integration/FeatureTree
Change-Id: I968f33f64f8683d00d9a49904e2aa74795267308
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
commit
fa2a2215478e20e31842338cb8a8bf919b220a33
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Mon Feb 15 22:35:40 2016 +0530
Bug 5351 Connection state tracking bit values are changed to match the
Ovs2.5 implementiaon.
Change-Id: Ia163223a7a25de3b7bca4360c3f6dbadd87851f5
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
92d8401231b863f1ff534c087bf6ff4844f20917
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Sat Feb 13 03:59:54 2016 +0530
Bug 4550
Changed the comparater to .equals.
Change-Id: Ie16cc681fc8e387cd2d85d097a3c380fd60f0e86
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
5b15fc7960aafd8200052945076abd517de7ae65
Author: Josh <jhershbe@redhat.com>
Date: Tue Feb 9 11:31:48 2016 +0200
BUG-4917 Ensure all OVS flows are deleted
This commit is basically a revert of this change:
https://git.opendaylight.org/gerrit/#/c/27521/
With that change the cache cleanup in fact never ran because the
port is removed before the code is actually run.
The bug for which that commit was a fix is resolved
with this code new commit as well.
In addition, updateL3ForNeutronPort() now allows
deleting flows even when networkId is not found
in networkIdToRouterMacCache. This takes care of
some of the DHCP issues.
Change-Id: Idc3c1ff4b57f5471615ccc6c627ca8f411c37db2
Signed-off-by: Josh <jhershbe@redhat.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit
5824899205c6eb36b45064a3a3f54668e1117a40)
commit
5aea3c01d6b54a0317d74bf50c45ed442b6a93df
Author: Stephen Kitt <skitt@redhat.com>
Date: Thu Feb 11 18:09:34 2016 +0100
BUG-5006: rework SouthboundMapperTest without PowerMock
A number of tests were meaningless, these have all been fixed to
verify SouthboundMapper's behaviour.
Conflicts:
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/SouthboundMapperTest.java
Change-Id: Ia349e545bde9706d2b1840530abd46c09ac55e45
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
commit
9fbb4ee65615591a6fafc729f2d13687e4beeb60
Merge:
1432633 aeef2f8
Author: Sam Hague <shague@redhat.com>
Date: Fri Feb 12 00:37:05 2016 +0000
Merge "Clean up SouthboundMapper (and fix createQosType())"
commit
1432633224e4241b226f92ef194b7d93a3825d50
Merge:
a8cb2f9 63a093b
Author: Sam Hague <shague@redhat.com>
Date: Fri Feb 12 00:36:28 2016 +0000
Merge "Bug 4550 - always rebuild security group rule in security group"
commit
a8cb2f94dce3d31455d4c7a97d2296fdd93f76f8
Merge:
5a54a87 73045d9
Author: Sam Hague <shague@redhat.com>
Date: Fri Feb 12 00:01:22 2016 +0000
Merge "Bug 4917 For fixing SG dangling flows. Extended port cache logic to TenantNetworkManagerImpl and SouthboundHandler."
commit
63a093bab0fca4500e26dc4db534dce3d279cc6d
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Wed Feb 3 08:02:52 2016 -0800
Bug 4550 - always rebuild security group rule in security group
The change set of
906836c289a7e4c3d33669d90515d77e75cfb6d in
neutron northbound repository, the effort transparent neutron
northbound api, caused not to update security group rule in
security group.
So always rebuild rules in NeutronSecurityGroup and keep
security group rule in security group of yang model empty list.
openstack service provider that uses MD-SAL should not use
security group rules in security rule. But it should listen to
data change notification on security group rules.
Change-Id: Ic975e028c01462a2c7267f65c16b3bd857496c73
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
(cherry picked from commit
a18e3f5a3744392c4475c82e4937af2a27d729fa)
commit
73045d9cf43b02ecdb6826d2efe9bdee211e322a
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Thu Feb 11 06:52:42 2016 +0530
Bug 4917 For fixing SG dangling flows. Extended port cache logic to
TenantNetworkManagerImpl and SouthboundHandler.
Added cache for NeutronNetwork. Changed the portcLeanUpCache from set to
map as the entries were not getting cleaned up from the cache.
Change-Id: Id02f2ff85d3d67845d8b618fbcdfded1f7958553
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
(cherry picked from commit
c619afb3a9f8d93b05225daf0b257c28079d4216)
commit
5a54a8730758b855f71c188274213c358afa94f5
Author: Victor Pickard <vpickard@redhat.com>
Date: Wed Feb 10 08:55:40 2016 -0500
BUG 5164 Cleanup Floating IP Rules when VM is deleted.
Patch set 3: Address review comments.
Patch set 2: Fixed unit test.
When a VM is deleted, Neutron does not send a notification
to update/delete any floatingip that may have been associated
with a port. As a result, openflow rules for the floating ip
are left installed.
Neutron team says this is expected behavior for floating ip.
So, to cleanup, use neutron port delete event, and find
any associated floating ip, and cleanup installed OF
rules.
Change-Id: If9e2ffc5b16f0da636b3e9b6b1c4fbfd46fd6aff
Signed-off-by: Victor Pickard <vpickard@redhat.com>
commit
aeef2f8f89f06c3030cfaf4b2c1cf82a4f9de28d
Author: Stephen Kitt <skitt@redhat.com>
Date: Thu Feb 11 18:01:34 2016 +0100
Clean up SouthboundMapper (and fix createQosType())
SouthboundMapper has a few unused imports and one unused method; this
patch removes them.
It also fixes the default QoS type returned by createQosType().
Change-Id: Ic664ee98b9c4f99d8c28b1b1071837c352e63f81
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
2a375e18f988a33de5e1c1ac248b1062c3182f0a
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Wed Feb 10 14:15:40 2016 -0800
Bug-5040 : LLDP Spoofing attack warning when using Openstack with ODL Cluster
Change-Id: Ife9fa1bd8fcd02ca4af67c71d2d3429f6477310f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
58d6c90a727623c1fb1726bbaaf1075f4a92fc02
Author: Eric Multanen <eric.w.multanen@intel.com>
Date: Wed Feb 3 17:00:56 2016 -0800
OVSDB Qos and Queue - Postman examples
Adds a Postman collection of RESTCONF commands for doing
CRUD operations with Qos and Queue entries.
v2 - remove tabs
v3 - remove tabs in right file
Change-Id: I8e98b2f9b5c6e517a5a60516fe21fce063e169e2
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
commit
69a9644e1dadbf8da1fe39b48fad9bf73fd9893d
Author: Sam Hague <shague@redhat.com>
Date: Thu Feb 4 18:17:28 2016 -0500
Bug 4996 - Wrong flows when using SFC coexistence
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit
e0bf1166f4ad2515859ab43080a28115975a2cf5)
Change-Id: I0874c752f8777d8a3f6a7f8f60db22c605c986a3
Signed-off-by: Sam Hague <shague@redhat.com>
commit
113f93d63d1b69e30a5606f7545221215cd85a3d
Author: Sam Hague <shague@redhat.com>
Date: Wed Feb 3 21:26:56 2016 -0500
Bug 4996 - Wrong flows when using SFC coexistence
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit
498fe4df2997839fdf12555eb5961008ce9eab1a)
Change-Id: Ibf950ae9b56317441014497ffd4260580326d802
Signed-off-by: Sam Hague <shague@redhat.com>
commit
6e564ab34d90933f9a8642019af87a5705eb692b
Merge:
c9ca37d 8653138
Author: Sam Hague <shague@redhat.com>
Date: Thu Feb 4 14:13:12 2016 +0000
Merge "Fix for changes to neutron-l3.yang"
commit
c9ca37da20f4c01214679b26d951ea5feb8cfd03
Merge:
1a5aa59 a0279e8
Author: Sam Hague <shague@redhat.com>
Date: Thu Feb 4 02:42:46 2016 +0000
Merge "BUG-5006: rework tests using AbstractDataBrokerTest"
commit
1a5aa59f6bf9d39f00225e292cb8fef3f818b778
Merge:
e035ab6 7e0433e
Author: Sam Hague <shague@redhat.com>
Date: Thu Feb 4 02:42:27 2016 +0000
Merge "Use constants for NeutronSecurityRule"
commit
8653138005a4ac53430c1f205680b3d9870b128c
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Wed Feb 3 00:23:25 2016 +0530
Fix for changes to neutron-l3.yang
Fix for BUG5137 in neutron breaks any code using routes option of
NeutronRouter. This fix is to change type of routes from String to Routes
as per yang changes.
Change-Id: I89866b31003a8b384c45ef9b451c5d189961df60
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
(cherry picked from commit
61b1b633dfcd9fc1080286ef32d5683e41dff5a5)
commit
e035ab6b71bd1528cb12dc0f87663359d2c11c88
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Wed Feb 3 03:06:44 2016 +0530
Fixes Bug 5187
remote security group will continue the addition/deletion of rules after
logging if not found in cache.
Change-Id: Ic6129b0e8884a185e0907789de66a5df3eee39c0
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
a0279e88fb94bd45a9ce5a50994bfd76fd3e8eb6
Author: Stephen Kitt <skitt@redhat.com>
Date: Tue Feb 2 14:33:23 2016 +0100
BUG-5006: rework tests using AbstractDataBrokerTest
Change-Id: I5d7acd00cfc71291e622bd79dc850a24be63f084
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
faf5434c57199811a1f30e30b7e610324e0e0dce
Author: Rashmi Pujar <rpujar@inocybe.com>
Date: Mon Feb 1 13:05:41 2016 -0500
BUG 5172: Handle SchemaVersionMismatchException for lldp column
Change-Id: I3014048642b298bdb347660c3ac0704c23996e22
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
commit
7e0433e7dc4d4ffd3ed0ad50ab728484c35b8fae
Author: Stephen Kitt <skitt@redhat.com>
Date: Mon Feb 1 17:39:43 2016 +0100
Use constants for NeutronSecurityRule
This patch introduces constants for the directions, protocols and
ethertypes in NeutronSecurityRule.
There's also a bunch of small clean-ups in the affected classes.
Change-Id: Ia07efa48eb431f560d9f1f947170e1fb68e24ddb
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
0204e4324aa972983c52a7a81d64c45dfb4bb26c
Author: hari.pr <hari.pr@hcl.com>
Date: Tue Jan 26 11:26:45 2016 +0530
BUG-4892:fixed Not getting Arp flows if there is no Router
*Moved the ARP handling part to a new file, This way the
ARP Entries are not restricted to l3 operations only.
*Tested L2 and L3 operations.
From our understanding
In router scenario whenever router created using openstack, netvirt
south bound handler will receive interface update event. ARP responder
will program the arp rules for the router interface event through
distributed arp service class.
(openstack àrouter create àsouthbound handleràprocess event àprocess interface updateàARP handleinterface eventàprogram ARP)
In normal L2 scenario whenever tenant network created using openstack,
netvirt will receive neutron port add event distributed arp service
class use this neutron port details to program arp rules for the DHCP IP.
Whenever VM instance created using openstack, netvirt will receive
neutron port event distributed arp service class use this neutron port
details to program arp rules for particular VM IP.
(openstackàVM instance à port handleràprocess eventà do neutronport(ADD/UPDATE/DELETE) à handle port àprogram ARP)
Change-Id: Ie56bd5d407f501b8a82f9fe3ad0672e57448eeef
Signed-off-by: hari.pr <hari.pr@hcl.com>
commit
9d0046e3ae85b596c8ae9cb10f8abda09eb40114
Merge:
91646ba 9ac9260
Author: Sam Hague <shague@redhat.com>
Date: Mon Feb 1 15:48:42 2016 +0000
Merge "BUG-5110 Enable pinging any router"
commit
91646badd0f066087774599e9cc002e9ff23bcdd
Merge:
413cf2f 17998ad
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 31 22:08:37 2016 +0000
Merge "Bug 5149: Support LLDP on ovsdb interface"
commit
17998adf6504880b2f255e514e1335527a166b4f
Author: Rashmi Pujar <rpujar@inocybe.com>
Date: Fri Jan 29 17:22:50 2016 -0500
Bug 5149: Support LLDP on ovsdb interface
Change-Id: I9b61d2e149479ae95f6978ae04e4f4e3e2a6df53
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
commit
9ac9260ad30a84f52ad120ef9f4192e0526be8ad
Author: Josh <jhershbe@redhat.com>
Date: Wed Jan 27 18:08:12 2016 +0200
BUG-5110 Enable pinging any router
Works like this:
In table 60, routed packets have their src MACs
saved to reg4 and reg5
In table 70:
ICMP echos are responded to, for routed packets
and for pings to routers on the same LAN
Change-Id: I23e3a6ef5397aa687e14745841efeccc3f7a5301
Signed-off-by: Josh <jhershbe@redhat.com>
commit
413cf2fbe170edbbfa919ba009a6dc57bf4ab9be
Merge:
7c887fb c45172f
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 31 01:43:56 2016 +0000
Merge "Add IPv6 SecurityGroup tests."
commit
c45172f9700c7fff3cb50a34312cdbb981e5e96d
Author: Sridhar Gaddam <sgaddam@redhat.com>
Date: Wed Jan 27 18:27:40 2016 +0530
Add IPv6 SecurityGroup tests.
This patch also includes fix for Bug-5134: Address exceptions when SG
remote_group_id has both IPv4 and IPv6 addresses.
Change-Id: I25855fa0bd5fe1938efb68a459982d790f4bdb27
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
commit
7c887fbb330b840276af8c4e228fcbb0cfb3d73a
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 30 11:55:36 2016 -0500
Bug 4996 - Wrong flows when using SFC coexistence
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit
d0a7d318ebf445da858aee508b477d0584977fa6)
Change-Id: I6c54b206cd9653868d16f1799ff21891af6333da
Signed-off-by: Sam Hague <shague@redhat.com>
commit
aec160cf0fe8772e51807c4fc111dc56bb1ef26a
Author: Sam Hague <shague@redhat.com>
Date: Fri Jan 29 15:20:52 2016 -0500
Bug 5147 - Wrong logging level for ConfigProperties not found with defaults
Change-Id: Ib53ef62942c4248fc258d752aea5f570447312c3
Signed-off-by: Sam Hague <shague@redhat.com>
commit
46341438eac4b66fb6ac836e3c35d2270540fe5c
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 28 16:51:51 2016 -0500
Bug 4996 - Wrong flows when using SFC coexistence
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit
f00b01e77a14b1ad53b6553296499c9d30365808)
Change-Id: I25fd28e89f78fb101c1e187d0ce06eba02567dfb
Signed-off-by: Sam Hague <shague@redhat.com>
commit
7929d631bad65da4f846a7cf696f0416f21c91d3
Author: stack <stack@localhost.localdomain>
Date: Tue Jan 19 14:41:29 2016 -0800
Unit tests for OVSDB clustering
Added some unit tests to OVSDB Clustering work done
during 'Be'.
V2: Removed unit test for deleteNode as this method was unused.
Change-Id: Icd18b87cd472d06317a5a7368acfc19c66f1cef4
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
commit
780548ec348a0c48920b4352955d404d0f63dfe0
Merge:
4881ddf 52d58be
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 28 15:37:08 2016 +0000
Merge "[Bug 5131]- Hard-coded base url in Ovsdb-ui"
commit
4881ddf44d9257756ca503d39c29c88a5019e711
Merge:
c1e8cd1 0a6c4a7
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 28 15:36:33 2016 +0000
Merge "BUG-4794: use the transaction invoker for hwvtep"
commit
c1e8cd1ac6a480b1f2c96b25056ba2d9cc769122
Merge:
c3b1eb1 d76113e
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 28 15:35:26 2016 +0000
Merge "BUG-2625: remove unneeded equinox dependencies"
commit
c3b1eb1216f15d2ae5b0872639a53e5ee3c751c2
Merge:
4a58ce5 755a55d
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 28 15:34:40 2016 +0000
Merge "Remove SouthboundUtil::deleteNode()"
commit
52d58be729b48081e62aaeee5aa2a43e9361ddda
Author: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>
Date: Thu Jan 28 09:34:37 2016 -0500
[Bug 5131]- Hard-coded base url in Ovsdb-ui
- Localhost is hard-coded in Northbond handler, so make it dynamic.
- Use a empty gif has logo so you won't see a broken image and 404 error
log
Change-Id: Ie30c15b8e818ccec2038908af5ffb02a2b595a7a
Signed-off-by: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>
commit
0a6c4a758873dc88d332138b6b23addc916d0516
Author: Stephen Kitt <skitt@redhat.com>
Date: Thu Jan 28 10:51:38 2016 +0100
BUG-4794: use the transaction invoker for hwvtep
HwvtepConnectionManager can delete a connection without going through
the transaction invoker. This patch fixes this and removes the
now-obsolete deleteNode method from HwvtepSouthboundUtil.
Change-Id: I623016599a7de2cc7e6edf72865e10af452f3a8b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
d76113edd5e4303be73e40f34d7c5dc65ca5768f
Author: Robert Varga <rovarga@cisco.com>
Date: Wed Jan 27 22:27:04 2016 +0100
BUG-2625: remove unneeded equinox dependencies
These dependecies are not needed, remove them.
Change-Id: I4dccd9c3e080047b8d958e201ddf0cc84c78f2a6
Signed-off-by: Robert Varga <rovarga@cisco.com>
commit
4a58ce5c3e152946c34cfc7344c0314beb3ef76b
Author: Stephen Kitt <skitt@redhat.com>
Date: Wed Jan 27 14:49:24 2016 +0100
Fix generics for JDK 8
Change-Id: I473fcee5429f9b7736b175ed4633cc9d7dafded2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
755a55d65026cb4efe80131d2e17f8398cb62ca8
Author: Stephen Kitt <skitt@redhat.com>
Date: Wed Jan 27 13:41:53 2016 +0100
Remove SouthboundUtil::deleteNode()
This method is unused and, since it doesn't use the transaction
invoker, potentially dangerous.
Change-Id: Ibbc674a5983d8e52094888a58eb6c7382daac467
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
87ea456c80503314fbac8a8e6fee3cd853b38288
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Tue Jan 26 15:44:06 2016 -0500
Bug-5069 Disable ovs in-band mode: cosmetics remove redundant line
Change-Id: Ibdf949f1aeff25fe4f69d5fe7f04bcabb7392c68
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
555b7f2e4836ec26acfab1288033e69a4beec833
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Tue Jan 26 13:53:04 2016 -0500
Boron: remove dependencies for neutron::dummyprovider
Now that neutron northbound project is no longer
requiring a neutron provider to exist [1], there is no
need to depend on that dummy feature. :)
[1]: https://git.opendaylight.org/gerrit/#/c/27291
Change-Id: Iedbb7253183f1830b64a2aa6ff0c5c5f498135f8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
1ad8f0727b92d2ca2ab53d1d530310d2c86aa2a6
Author: Stephen Kitt <skitt@redhat.com>
Date: Tue Jan 26 15:28:38 2016 +0100
BUG-4794: use the transaction invoker to queue the clean-up
When a manager disconnection comes in, an OvsdbRemoveNodeCommand is
queued in the transaction invoker to remove the corresponding nodes.
If an ownership change is detected, an "emergency" clean-up is
executed, but without going through the transaction invoker; it's
executed using the MD-SAL data broker directly. Both of these
operations try to protect themselves by checking the presence of
appropriate nodes before running; but in some cases we see
manager-node-entry list item deletions being attempted on a deleted
manager-node-entry list, which causes a crash.
To avoid the race which leads to these crashes, this patch queues the
emergency clean-up with the transaction invoker, ensuring that the
operations will be serialised with any other clean-ups. In this way
the protection checks will be effective and avoid deleting items from
no-longer-existent lists.
Change-Id: Ia4ff31677e61d897720ce3209d6da7eed5e5ee8d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
3993b5bcc79f28d73a7cfbc7b32e9c976a68b450
Merge:
48265a7 d64c55d
Author: Sam Hague <shague@redhat.com>
Date: Mon Jan 25 15:00:04 2016 +0000
Merge "Bug 5008 - QoS and Queue fixes for ovsdb southbound"
commit
48265a7b5389832cae2345a8faa97633917feb6b
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Mon Jan 25 22:10:17 2016 +0530
Fixes Bug 4643
The fix for the bug 4205 is extented to instances where neutron
port cache is used.
Change-Id: I5e3e90f2e3335e65dca29112a551ccd053823f53
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
ef59c766d692d5217b7d9a51dc28449e8a5652eb
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Mon Jan 25 21:52:20 2016 +0530
Fixes Bug 4769
The fix for the bug 4205 is extented to other instances where neutron
port cache is used.
Change-Id: Ic69142c1804a37725a9dc9c611d583302284d18c
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
9310aa02b740e1ab6156cd86f7579757fe2f49ff
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 23 17:35:07 2016 -0500
Bug 5066 - ConflictingModificationAppliedException: Node was created by other transaction.
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit
ef9d4d8fdd25d26d8d13b9d69e269a2e4becbbd6)
Change-Id: I175b007213908abd41f7a1168ad160fc86d337e9
Signed-off-by: Sam Hague <shague@redhat.com>
commit
d64c55d65dda23f61737086d1522b0c2347b2391
Author: Eric Multanen <eric.w.multanen@intel.com>
Date: Mon Dec 14 07:10:10 2015 -0800
Bug 5008 - QoS and Queue fixes for ovsdb southbound
Update the operational MD-SAL for Queue and Qos updates from the OVS
host.
Support configuration of QoS and Queue via configuration MD-SAL.
- Support for adding and deleting QoS entries. A qos-id is used
as a handle for qos entries in the MD-SAL. This is added to
the 'external-ids' of the qos entry. Qos entries originating
from the OVS host are assigned a qos-id of "qos://<uuid>".
- Qos entries can be assigned to a Port (termination point)
which is present in the MD-SAL by setting the port qos
attribute to the UUID of the relevant Qos entry.
- Qos entries are cleared from a port by updating the
port configuration with the qos attribute removed.
- Support for adding and deleting Queue entries. a queue-id
is used as a handle for queue entries in the MD-SAL. This
is added to the 'external-ids' of the queue entry. Queue
entries originating from the OVS host are assign a qos-id
of "queue://<uuid>".
- Queue entries can be added or deleted to the queue list
of a Qos entry present in the config by filling out (or
removing from) the queue-list attribute of the qos entry.
v2 - added setting of exising qos uuid to an existing port
- added clearing of a qos setting for a port
v3 - ovsdb.yang whitespace fixes
- fixes for failed tests
v4 - fixed model - queue list in Qos entries needed
a queuenumber key value
- fixed up comments on whitespace and log messages
v5 - finished set/clear of queues list in a qos entry
- added more description about the functionality in
the patch set.
v6 - fixed up more whitespace, and copyright comments.
v7 - address review comments:
- white space
- remove unnecessary code
- remove commented out code
- adjust some LOG levels
- remove unused imports
Change-Id: Idec02aaa82e1ffa66e58862ecfc5edff05050444
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
commit
c0f6a6e9243efc88eaeeecef2a71e611e8137e51
Merge:
e3936ff f89089e
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 23 22:48:37 2016 +0000
Merge "SouthboundIT: make "value mandatory" a builder property"
commit
e3936ff044b409a75777dec9c220725b19faff95
Merge:
fa0825c 1e7875b
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 23 21:32:02 2016 +0000
Merge "IPv6 support in Security Groups."
commit
fa0825c8cd467aee1f71fb9916ac7b5865df3fd8
Merge:
ec835d7 99c1b74
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 23 21:30:14 2016 +0000
Merge "Fixed UT and cleaned up code. 1. Fixed the UT issues in IngressAclServiceTest and EgressAclServiceTest. 2. Cleaned up unused method and UT in IngressAclService and EgressAclService."
commit
f89089ea91db55138c2df3ac3bba8a52af60a4cd
Author: Stephen Kitt <skitt@redhat.com>
Date: Fri Jan 22 10:57:05 2016 +0100
SouthboundIT: make "value mandatory" a builder property
Instead of coding explicit instanceof tests, turn the mandatory nature
of the value into a builder property.
Reset the builders immediately after use.
Change-Id: I5fc20a17ff903cb01a0b8c091300c099c84ab3e4
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
ec835d7604572c4c96d71872495628c4703bbcfc
Author: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Fri Jan 22 00:50:27 2016 -0500
Re-enable net-virt-sfc
Change-Id: Id54c96b226b832db5d91987fce8d549a35ad228a
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
commit
1e7a6189d8b518b4377ec697a300d5a73e36f3d4
Author: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Thu Jan 21 15:51:40 2016 -0500
Bump yangtools to 1.0.0-SNAPSHOT
Change-Id: I1c28f5240c202bc8d79e0c0ecd3ae84e7582c00b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
commit
99c1b7476543e522fa13e56ad79590779b04c09a
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Fri Jan 22 02:52:48 2016 +0530
Fixed UT and cleaned up code.
1. Fixed the UT issues in IngressAclServiceTest and
EgressAclServiceTest.
2. Cleaned up unused method and UT in IngressAclService and
EgressAclService.
Change-Id: I8632eb28ae292ea7a50968291cf9f6bdfc92fdaf
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
1e7875bb285d3953057816937311eee19b627b5b
Author: sridhargaddam <sgaddam@redhat.com>
Date: Mon Jan 11 21:40:49 2016 +0530
IPv6 support in Security Groups.
This patch implements IPv6 support for Security Groups in OVSDB
Net-virt.
Pending: Test cases will be added in a subsequent patch.
Change-Id: If692c9b205af87517d69cbcc9f8c1be4decc917e
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
commit
05213e174213ea961e0ca2e5a330078c587a9d65
Author: Josh <jhershbe@redhat.com>
Date: Tue Jan 19 18:02:43 2016 +0200
BUG-5013 Fix check logic in MatchUtils.createICMPv4Match
The original logic checked that *either* of the
input params was non-null instead of checking
each.
Change-Id: I941133c00b09335eb0cf4a48b9abf35d8b305438
Signed-off-by: Josh <jhershbe@redhat.com>
commit
0289b397c733c6ad0146f3fa9251433c7e3e49a6
Merge:
6d79d6a 60dcda6
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 19 13:14:49 2016 +0000
Merge "Bug 5007 - Reenable SG IT"
commit
6d79d6af2a8a3fce1505a3ca0c625d7bfecfa4af
Merge:
821ae53 0d77ee1
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 19 11:25:01 2016 +0000
Merge "BUG-5006: Rework testEnqueueEvent to avoid private methods"
commit
821ae5392a4a5efc3ef09b933391e050975fcc41
Merge:
06a05f7 2fbc8d7
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 19 11:24:40 2016 +0000
Merge "BUG-5006: Reduce usage of PowerMock"
commit
60dcda67d6823180faddaff41809f2763254ff42
Author: raveek <raveek@hp.com>
Date: Tue Jan 19 16:19:58 2016 +0530
Bug 5007 - Reenable SG IT
Change-Id: I5526fcc356b58484613d9dfcbfa453d66c0e354a
Signed-off-by: raveek <raveek@hp.com>
commit
0d77ee109e4687c06c4bc977215aa27c22a8a4d2
Author: Stephen Kitt <skitt@redhat.com>
Date: Mon Jan 18 18:00:55 2016 +0100
BUG-5006: Rework testEnqueueEvent to avoid private methods
Instead of checking calls to dispatchEvent(), provision an event
handler and use that to make sure the queue is processed. Stop the
event processing thread to avoid races in the assertions.
Change-Id: Ic385314e15dead0c1f796c25863136a3a60099ac
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
2fbc8d731a29e0a9fd49ba0dcecee379e6aa9e0b
Author: Stephen Kitt <skitt@redhat.com>
Date: Thu Jan 14 16:56:26 2016 +0100
BUG-5006: Reduce usage of PowerMock
Make ServiceHelper and ConfigProperties overridable; this allows most
of the tests using PowerMock to switch to plain Mockito.
Change-Id: I50c5c64f687fb0b0aa73194fb3d6266951eedb23
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
06a05f7200e065c697c07d991f9b255a40fffa11
Merge:
7d4a072 4bc8c49
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 19 00:40:04 2016 +0000
Merge "Bug 4996 - Wrong flows when using SFC coexistence"
commit
7d4a072d88d0e23d64765886783a7967096bcd1c
Author: Sam Hague <shague@redhat.com>
Date: Mon Jan 18 16:59:59 2016 -0500
Bug 4997 - Fix for ConnectionInfo parsing in showMdsal tool
Change-Id: If3fee23373829c520fcc532ba70b1a45061aa2b3
Signed-off-by: Sam Hague <shague@redhat.com>
commit
4bc8c49028b37e09719106e761de71f1f9720e03
Author: Sam Hague <shague@redhat.com>
Date: Mon Jan 18 16:58:42 2016 -0500
Bug 4996 - Wrong flows when using SFC coexistence
Change-Id: Icb9cab27b271f0e5462ecb87ead7a10f0f083bd1
Signed-off-by: Sam Hague <shague@redhat.com>
commit
cd2ba3039ca0174a75a6879e2bf9ea0261c3ac53
Merge:
11a67c2 cee306e
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Fri Jan 15 14:58:48 2016 +0000
Merge "BUG-4205 : VM delete doesnot remove all related flows"
commit
11a67c2a846e7cd538ace70e9a868443995a05e5
Merge:
3e8f5e7 fc81e82
Author: Sam Hague <shague@redhat.com>
Date: Fri Jan 15 14:46:52 2016 +0000
Merge "Fixes an issue in induvidual security rule update, when remote SG is selected."
commit
cee306ea08d5d3c2bd5d56679cb681edc90de913
Author: bviswa <badrinath_viswanatha@dell.com>
Date: Thu Oct 22 08:38:01 2015 +0530
BUG-4205 : VM delete doesnot remove all related flows
Patch set 3: Rebase and add ref to bug 4971
Patch set 5: Rebase again and move processSecurityGroupUpdate()
Change-Id: Ia3cbba3087232cb56eee6e5424cdd011a01ce266
Signed-off-by: bviswa <badrinath_viswanatha@dell.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
3e8f5e701be9b8ac5ea288d6c644b85661571803
Author: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Fri Jan 15 01:33:11 2016 -0500
Bump versions by 0.1.0 for next dev cycle
Change-Id: I647f1e840b4d4bc587f6cbce4018ed187922a7a0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
commit
fc81e82d1447ffc8e8f4b20019a9e4251243805c
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Fri Jan 15 01:14:01 2016 +0530
Fixes an issue in induvidual security rule update, when remote SG is
selected.
Change-Id: Id8d5ea6c6235c31f9147556244f619a15a279382
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
146664effb4293306981a8fe6f6409b925c4fdab
Merge:
5199869 b44a82f
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 14 11:10:30 2016 +0000
Merge "Add support for bfd-status for monitoring"
commit
5199869f6edca82c8a8d5e0696397dbaff3b5bef
Merge:
9039d23 de9497b
Author: Sam Hague <shague@redhat.com>
Date: Thu Jan 14 03:19:22 2016 +0000
Merge "Workaround for failing Netvirt SG IT"
commit
de9497b9f6cbf0c32752e2113ea2cfb135381c60
Author: Sam Hague <shague@redhat.com>
Date: Wed Jan 13 21:01:23 2016 -0500
Workaround for failing Netvirt SG IT
Change-Id: I2e9ed58680ebefb6ceed912af5518b5eb543da7d
Signed-off-by: Sam Hague <shague@redhat.com>
commit
9039d23ee28862366e969f9975f2aa4941ccd1ba
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Jan 11 10:46:42 2016 -0800
Added cluster aware mdsal utils class to net-virt (redo)
Looks like while code cleanup we removed the MdsalUtils.java from the master branch
and while merging master branch to net-virt-clustering, it got removed as well.
Net-virt-clustering had a modified version of MdsalUtils.java, where put/merge/delete
transactions are only done by the master instances of net-virt. ClusterAwareMdsalUtils
class bring those changes back.
Patch set 2: Explicitly callback into OvsdbInventoryService when netVirt becomes owner.
Patch set 3: Undo patch set 2; make ProviderEntityListener to happen earlier
Change-Id: I42ab5f2969bcff7737d90f2651d2762897e74c9a
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
b2096a5af7375360917cedcede1101a3aa9bf5b7
Merge:
8b6789c 60a8068
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Thu Jan 14 01:11:29 2016 +0000
Merge "Revert "Added cluster aware mdsal utils class to net-virt.""
commit
60a806823592eb2137aa4c7221c368a6cbaee507
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Wed Jan 13 21:34:43 2016 +0000
Revert "Added cluster aware mdsal utils class to net-virt."
This reverts commit
e9ec56b353a2d9098e77d623b7c756069bb58cb9.
This causes a regression in that netvirt:1 is never written to mdsal. to be re-visited.
Change-Id: I683dcac86c23445762d460183fa3e651a600f2c2
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
8b6789c56441f347bcd3b5e3e22efb3246ea2357
Merge:
e9ef365 9f6c1c4
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Wed Jan 13 12:24:12 2016 +0000
Merge "Drop testRegisterNewInterface() entirely"
commit
b44a82f4b9378db48783698aa602de1b7ec1d021
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Wed Jan 13 12:40:29 2016 +0100
Add support for bfd-status for monitoring
bfd-status gives information about current state of tunnels. This required
to monitor tunnels and know when they go up/down.
Change-Id: I5d864d46adf1444b00382acef58e298df5dc155e
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
9f6c1c422c4c375277e3314faceac138a7c53bfd
Author: Stephen Kitt <skitt@redhat.com>
Date: Wed Jan 13 10:55:54 2016 +0100
Drop testRegisterNewInterface() entirely
This test requires a bundle context to work properly, so it's not
really doable in a unit test.
Change-Id: I29a5a74597146dc3342c6fe7906a28fe89d6d22a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
e9ef36579d56bf1a0f30f8aa9fb0bc97589012df
Author: Josh <jhershbe@redhat.com>
Date: Sun Jan 10 13:13:01 2016 +0200
Bug 4132 config ovs to respond to pings to router
Relies on https://git.opendaylight.org/gerrit/#/c/32135/
N.B. I added a unit test, IcmpEchoResponderServiceTest,
however it's currently skipped along with all the other
unit tests under services.
Patch set 8: Fix imports
Change-Id: If5d212c29acaf49d3cdd16cb9a02a63073202a88
Signed-off-by: Josh <jhershbe@redhat.com>
commit
d6db8266a0752bdfefe71d304a9b2ec0ba7767bf
Merge:
6322ebf e79de5d
Author: Sam Hague <shague@redhat.com>
Date: Wed Jan 13 00:23:23 2016 +0000
Merge "Use ClusteredDataTreeListener in hwvtepsb"
commit
6322ebf42f616a85604c7323c66e9d899dbe3b15
Merge:
c5e3722 7af0ce2
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 23:04:58 2016 +0000
Merge "Adding changes to use and support connecting tracking. By default SG will push flows with out conntrack match and action, but user can modify default-config.xml to enable conntrack."
commit
c5e3722c6e4018e87307e5d0a154ee42160c2d22
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 16:46:33 2016 -0500
Disable broken unit test: NeutronFloatingIPInterfaceTest
The test fails with:
Running org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest
Tests run: 9, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.908 sec <<< FAILURE! - in org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest
testRegisterNewInterface(org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest) Time elapsed: 0.054 sec <<< ERROR!
java.lang.NullPointerException: null
at org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterface.<init>(NeutronFloatingIPInterface.java:35)
at org.opendaylight.ovsdb.openstack.netvirt.translator.crud.impl.NeutronFloatingIPInterfaceTest.testRegisterNewInterface(NeutronFloatingIPInterfaceTest.java:329)
There are other exceptions listed earlier which look to be a conflict between the mockito and jacoco:
https://gist.github.com/
02f3a56db00ce9b15eb2
Change-Id: I519e63c4a8825d49189dbc4b241998b940c6f4f2
Signed-off-by: Sam Hague <shague@redhat.com>
commit
c20313b4a4fb6340d4291ce71ee37eea95bb7f7f
Merge:
d094926 697130c
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 20:06:22 2016 +0000
Merge "Enable PowerMock instrumentation with JaCoCo"
commit
d09492615ed3a81fbbeecb5c2d8e0632720a9644
Merge:
f21a2c0 e9ec56b
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 20:05:54 2016 +0000
Merge "Added cluster aware mdsal utils class to net-virt."
commit
f21a2c008227797911e7d1e244b07542e153e034
Merge:
e2529ab 582e717
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 20:05:28 2016 +0000
Merge "Explictly cleanup all the bridges managed by the node"
commit
e2529abd7d7496f477a0583bc03eac2bb1af8de5
Merge:
a60888a 6f82804
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 19:53:29 2016 +0000
Merge "Unit test file for NeutronFloatingIPInterface."
commit
a60888adee25fa56816bfb6dbeab4b8ef56e87df
Merge:
eda29ac 3be529d
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 19:52:53 2016 +0000
Merge "Bug 4927 - Rules added by MacResolverServices in br-ex are not removed"
commit
7af0ce240eb96245031675be2ed54a7de4f1a79e
Author: raveek <raveek@hp.com>
Date: Thu Jan 7 10:33:00 2016 +0530
Adding changes to use and support connecting tracking. By default SG
will push flows with out conntrack match and action, but user can modify
default-config.xml to enable conntrack.
Patch 2: after fixing the review comments
Patch 3: Fixed the test failure
Change-Id: I71d8fd23d66caad97e406507889ebba38bd80d27
Signed-off-by: raveek <raveek@hp.com>
commit
eda29ac5d45c822e69450f29d5d8eed93f370756
Merge:
f3c05d5 ed2a58b
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 19:50:08 2016 +0000
Merge " Added new feature *odl-ovsdb-openstack-clusteraware*."
commit
f3c05d54061fd99dc38a0c2e31e17d73ba074ec5
Merge:
2c8f9d8 364952c
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 19:49:21 2016 +0000
Merge "Add support for table offset"
commit
e79de5dbbe66b384a8b6aaa7f38515b3cd8780c2
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Tue Jan 12 20:30:53 2016 +0100
Use ClusteredDataTreeListener in hwvtepsb
Change-Id: I41234fd2ece1c618107041888d75dc9cc7a62e16
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
2c8f9d862821432c7e2fdbcaa5cb0614b3503680
Author: Sam Hague <shague@redhat.com>
Date: Tue Jan 12 11:35:41 2016 -0500
Fix npe from getLocalIp
Change-Id: I68336c2aa284bebb4a09d2fa596de91a1ee2d272
Signed-off-by: Sam Hague <shague@redhat.com>
commit
364952ce2fa3f452a9e1feebbf612394f023121c
Author: Sam Hague <shague@redhat.com>
Date: Mon Jan 11 14:57:15 2016 -0500
Add support for table offset
This commit adds a new config parameter table-offset to
allow setting the openflow table base. The default value is 0.
The user can change the value via the config file
netvirt-providers-impl-default-config.xml or by using the restconf
api show below. The value used must be between 1 and 145 - with 145
coming from 255-110 - 110 is the max table in the netvirt pipeline.
The following curl commands can be used to configure the value:
- Get the current value:
curl -u admin:admin http://localhost:8181/restconf/config/netvirt-providers-config:netvirt-providers-config
- Change the value: In this example the value is changed to 1
curl -u admin:admin http://localhost:8181/restconf/config/netvirt-providers-config:netvirt-providers-config -i -H "Content-Type: application/json" --data '{"netvirt-providers-config": {"table-offset": 1}}' -X PUT
If the table-offset is set, then an extra flow is added to the br-ex and br-int
switches to steer packets from table 0 to the first table in the pipeline.
IT tests were all updated to check for the new table-offsets as well as
adding unti-tests. Some IT tests have a little more due to the Yang
pacth that enforces strictre rules on the Yang files and was failing the tests.
Further patches should add config service in netvirt to be a central point
handling all the config updates. SG is also adding a config value that should
be included.
Change-Id: I44b3a442e8050d8f1ef1c5cd9db5c32af2d5ce19
Signed-off-by: Sam Hague <shague@redhat.com>
commit
697130c90da756caf7164313734496d3657f5ec7
Author: Stephen Kitt <skitt@redhat.com>
Date: Tue Jan 12 16:20:19 2016 +0100
Enable PowerMock instrumentation with JaCoCo
By default PowerMock is incompatible with JaCoCo, even with the older
version of JaCoCo defined by odlparent: the tests run but their
coverage isn't measured. This patch forces offline instrumentation
which allows both PowerMock and JaCoCo to operate (more or less)
properly. There are a number of "class already instrumented" errors
but they don't affect the test results.
This patch also upgrades PowerMock to 1.6.4 (initially in the hope
that it would work with JaCoCo as-is); this requires an upgrade to
Mockito too. I'll drive these changes properly in odlparent and remove
the local overrides when appropriate!
Change-Id: I2ab704f7a28f325ed3bb5a75a6f34ffb6c3b36b3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
e9ec56b353a2d9098e77d623b7c756069bb58cb9
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Jan 11 10:46:42 2016 -0800
Added cluster aware mdsal utils class to net-virt.
Looks like while code cleanup we removed the MdsalUtils.java from the master branch
and while merging master branch to net-virt-clustering, it got removed as well.
Net-virt-clustering had a modified version of MdsalUtils.java, where put/merge/delete
transactions are only done by the master instances of net-virt. ClusterAwareMdsalUtils
class bring those changes back.
Change-Id: I1d5353572c408e519d7a3cc75356b73436dae147
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
582e717190867d42b473896c0544bae580989301
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Jan 11 10:01:40 2016 -0800
Explictly cleanup all the bridges managed by the node
Change-Id: I37d608a7bb1096d7291d12a0d50dd482f257542f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
3be529d8fc7831297b68f0e8b0bc6425b3f7f6a2
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Mon Jan 11 20:58:58 2016 -0500
Bug 4927 - Rules added by MacResolverServices in br-ex are not removed
This bug can be reproduced by using an external router address
that is not owned by any system in the external neutron subnet.
Made the flow remove call idem-potent and in a common function.
Added missing flow remove calls in the following 2 missing places:
* Entry removal (non periodic and periodic)
* Time period restart
Change-Id: Idf481d3a13c1728bf400f185de296cbada7d920b
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit
b1cf5de78228b23a4c6aae1613df925acfa798c9)
commit
c4a0bde241862b3079c7be203690d6653386c8bb
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Thu Jan 7 22:57:33 2016 -0500
Bug 4331 - External bridge used for gateway mac resolver needs to be dynamic
With this change the GatewayMacResolverService will become resilient to the cases
where nodes get deleted or lose br-ex connectivity after the resolver is kicked
off to periodically track a given ip address.
Also made some minor clean up in NeutronL3Adapter, so that it needs not to
duplicate work for tracking ip in the very first iteration. That duplication
is not necessary as the GatewayMacResolverService is already implemented
with thread executors for the periodic monitoring.
Lastly, this gerrit augments the GatewayMacResolver service to provide
an extra boolean param called refreshExternalNetworkBridgeDpidIfNeeded.
That can be used to control whether dpid used gets updated when mac is
not getting resolved.
Change-Id: I95b9e5a2d5773a6be82221b71000ace2a662e772
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
715dc96a060b646cd87a827d45f0ecec68ff2e6f
Merge:
0c38b6e 1c9dc36
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 21:52:50 2016 +0000
Merge "Bug 4913 - populate openflow rules for security group on port creation"
commit
0c38b6e239976a5f19f3b10d6ae68552a83539f1
Merge:
ea3a1ad 7003972
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 21:50:22 2016 +0000
Merge "Bug 4912 - NeutronL3Adapter: missing/reordered enabled check"
commit
ea3a1ade85fc9552eb2a6255a385697e89c5dbdd
Merge:
b7a5a0c 99ec887
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 15:28:06 2016 +0000
Merge "Fix UT: call method under test and verify"
commit
b7a5a0c096440f47d478cbc07f8d9e944008b125
Merge:
62a1c58 9606248
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 13:08:12 2016 +0000
Merge "Bug 4881 - failure in netvirt.AbstractEventTest"
commit
62a1c582d451f0a30b6404d8449b1879b4c0c02c
Merge:
94aa9a3 c96da2c
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 13:06:26 2016 +0000
Merge "Bug 4911 - unbreak
a55db97e8ce43aec9e2f3a3fe70f6bec3272195b"
commit
94aa9a33be2a8b4f1f87caa4785912bc6c743bd5
Merge:
e7281ec 7aead48
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 13:00:43 2016 +0000
Merge "Add netvirt and hwgw features to main ovsdb/features."
commit
e7281ecd6f3f4c375727d41130e9902cd6daffce
Merge:
0e14c9c fe75573
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 12:59:59 2016 +0000
Merge "QoS Schema - fix type of QoS type"
commit
0e14c9cbe290954fc80d445be1eae085aedbf59d
Merge:
3b640b6 e348249
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 12:59:51 2016 +0000
Merge "Remove unused SchemaVersionMismatchException constructors"
commit
3b640b616ac74936fde2013721789f03094f39c1
Merge:
22222f3 dfc18e8
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 12:59:09 2016 +0000
Merge "Bug 4874 - distributed arp in old l3 for ovsdb is not installing rules when it should"
commit
22222f3cdfd17381b1e1c9b194e82b9f9fb16a69
Merge:
00594d7 d5bd15d
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 12:58:04 2016 +0000
Merge "Add a unit test for TyperUtils::checkVersion()"
commit
00594d7fc52cc75462d18162a9844138b7601078
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 9 22:37:46 2016 -0500
Remove failing SouthboundIT test because of new yang enforcement
https://bugs.opendaylight.org/show_bug.cgi?id=4355 added enforcement of values
marked as mandatory in a leaf. The failing tests were negative testing for
setting external_ids with an empty value. That test is invalid now that the
mandatory value is enforced.
Missed a instance in this patch: https://git.opendaylight.org/gerrit/#/c/32321/
but it was already merged.
Change-Id: I5fcc140907b4deec69e025a5055963f4a6743a01
Signed-off-by: Sam Hague <shague@redhat.com>
commit
cf5175afaaa96c83bcbc6b056795dce8576ae430
Merge:
c3ebb7d 3212466
Author: Sam Hague <shague@redhat.com>
Date: Sun Jan 10 03:40:25 2016 +0000
Merge "Disable hwvtep tests until impl is ready"
commit
c3ebb7d7dac1bb8e3352816358ae9d8772f48aa8
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 9 20:56:54 2016 -0500
Remove failing SouthboundIT test because of new yang enforcement
https://bugs.opendaylight.org/show_bug.cgi?id=4355 added enforcement of values
marked as mandatory in a leaf. The failing tests were negative testing for
setting external_ids with an empty value. That test is invalid now that the
mandatory value is enforced.
Change-Id: If6cdcd8a899a6fcde5ca00a2ad2c505e728499e9
Signed-off-by: Sam Hague <shague@redhat.com>
commit
6ef6443fc0f6448439f856c415f515db0a0f47c8
Merge:
05dbf69 7f1836f
Author: Sam Hague <shague@redhat.com>
Date: Sat Jan 9 15:42:25 2016 +0000
Merge "Disable hwvtepsouthbound-it"
commit
1c9dc3604a5ba49aa01b86dce705f202b2d8cd1f
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Fri Jan 8 17:45:53 2016 -0800
Bug 4913 - populate openflow rules for security group on port creation
When port is created, original port is null.
In that case, flow should be populated instead of ignoring the events.
Change-Id: I1f90868bbb4bcb3e7d40474133bcaafe2fc1983a
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
commit
7003972d8cadaa2e73412d02640153ca70319314
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Fri Jan 8 18:05:40 2016 -0800
Bug 4912 - NeutronL3Adapter: missing/reordered enabled check
NeutronL3Adapter.processEvent() can be called before its initialization
The security group process should be after enabled check.
Change-Id: I7fc48a70cacca5a9a1d7a017476259543c8a0086
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
commit
c96da2ce6ad9fc7e9cced9c9968104b25b83fd20
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Fri Jan 8 18:15:49 2016 -0800
Bug 4911 - unbreak
a55db97e8ce43aec9e2f3a3fe70f6bec3272195b
The change set of
a55db97e8ce43aec9e2f3a3fe70f6bec3272195b,
change id of Ib740544ddbbb08745aa7f21aa9246acad1ad1d53
breaks ipv6 work around as follows.
This patch unbreaks it and apply same change to similar work around.
> java.lang.IllegalArgumentException: Could not parse [2003:0:0:0:0:0:0:1/64]
> at org.apache.commons.net.util.SubnetUtils.calculate(SubnetUtils.java:240)[242:org.apache.commons.net:3.3.0]
> at org.apache.commons.net.util.SubnetUtils.<init>(SubnetUtils.java:52)[242:org.apache.commons.net:3.3.0]
> at org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.services.RoutingService.programRouterInterface(RoutingService.java:62)[279:org.opendaylight.ovsdb.openstack.net-virt-providers:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programRouterInterfaceStage2(NeutronL3Adapter.java:1195)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programRouterInterfaceStage1(NeutronL3Adapter.java:1183)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programFlowForNetworkFromExternal(NeutronL3Adapter.java:1080)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.programFlowsForNeutronRouterInterface(NeutronL3Adapter.java:1054)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleNeutronRouterInterfaceEvent(NeutronL3Adapter.java:522)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleNeutronPortEvent(NeutronL3Adapter.java:483)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.NeutronL3Adapter.handleInterfaceEvent(NeutronL3Adapter.java:796)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.handleInterfaceUpdate(SouthboundHandler.java:91)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processPortUpdate(SouthboundHandler.java:299)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processPortEvent(SouthboundHandler.java:287)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.SouthboundHandler.processEvent(SouthboundHandler.java:232)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.dispatchEvent(EventDispatcherImpl.java:95)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl.access$200(EventDispatcherImpl.java:27)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at org.opendaylight.ovsdb.openstack.netvirt.impl.EventDispatcherImpl$1.run(EventDispatcherImpl.java:56)[278:org.opendaylight.ovsdb.openstack.net-virt:1.2.1.SNAPSHOT]
> at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)[:1.8.0_66]
> at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0_66]
> at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)[:1.8.0_66]
> at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)[:1.8.0_66]
> at java.lang.Thread.run(Thread.java:745)[:1.8.0_66]
Change-Id: I979f7538c9eb8686e9de2773e70514dfb90a4892
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
commit
ed2a58bcf272b751d3f940277f78c53dc0d524be
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Fri Jan 8 17:47:51 2016 -0800
Added new feature *odl-ovsdb-openstack-clusteraware*.
This feature loads all the required component to make
net-virt cluster aware, like openflowplugin-*-li
features that support clustering
NOTE: THIS FEATURE DISABLE THE AUTO LOAD OF
odl-ovsdb-openstack AT BOOT TIME. USER NEED
TO EXPLICITLY INSTALL THE REQUIRED FEATURE.
Change-Id: Ie087ad3d8b805838125e25d42d87e1ffb1758414
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
05dbf69a2b11ebdf586aeed935497266a5c3c933
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Fri Jan 8 16:15:19 2016 -0800
Merge 'topic/master/net-virt-networking' to 'master'
Squashed commit of the following:
commit
11abd792240e8df1a51a012ba1b92dd7c920c38b
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Fri Jan 8 16:11:50 2016 -0800
Revert to old openflowplugin
Change-Id: Icaa1fa04fcc319b128e6f1ee0d58b940385812f3
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
83e3dab5337d20168280ba6018ed0b811c713314
Merge:
6801901 9484b0d
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Jan 4 17:14:55 2016 -0800
Merge branch 'master' into topic/master/net-virt-clustering
Change-Id: I4aa7a2d4d10df28f15edcedf5e7a74f16a439ab5
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
68019016506b78adef37f7c7764c0622f5bfb61e
Merge:
23295db 9b3005b
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Sat Jan 2 13:19:59 2016 -0800
Merge branch 'master' into topic/master/net-virt-clustering
Change-Id: I29e864f4c09862dc8e41110a77479e6ee6d0632a
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
23295dbb6f26fc0b24521949a14743ddad6d45fb
Merge:
a1c8e2b eced0bd
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Tue Dec 29 15:52:23 2015 -0800
Merge branch 'master' into topic/master/net-virt-clustering
Change-Id: Ib465c8f20f200d407caaa9440bbf635d72776fe1
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
a1c8e2b2a9738ae946731fb4d111f152057d3b71
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Tue Dec 22 15:39:00 2015 -0500
Remove extra return statement.
When Flow id caches that were no longer needed
were removed from code, a lonely return statement
in the router interface remove code path was
left behind.
Gerrit: https://git.opendaylight.org/gerrit/#/c/29204/
Change-Id: Ic978953ab65da91baf543e21ea86333b4a9ee5ca
Reference: Ic2acced7a9a006e6e04afba6fea22aaf0620f134
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
00de61c7b19937c5b48c1af4d50ae73cdb92b809
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Tue Dec 15 21:23:49 2015 -0800
Bug 4789 - allowed address pair doesn't have port id
So getPortId, setPortId methods aren't defined.
This is the ovsdb part of Bug 4789.
Change-Id: Id9ad4762b5ed92c5b84e6fab1d7c5135782ca528
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
(cherry picked from commit
2065f152079fb96f494518efc2b46f4c77db181d)
commit
08ba6afc0ae524716656e5e926bf34675a2e0960
Merge:
4b909d1 9bbb7db
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Dec 14 13:53:12 2015 -0800
Merge branch 'master' into topic/master/net-virt-clustering
Conflicts:
features/ovsdb/pom.xml
features/src/main/features/features.xml
openstack/net-virt-providers/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/AbstractServiceInstance.java
openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtils.java
openstack/net-virt/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtilsTest.java
Change-Id: I68702767c0fcae8cb9721805e524d3ecb7889ed4
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
4b909d144f06d4924792c42f918e1f0ce1a86137
Merge:
ddbb2eb 5483b28
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Fri Nov 20 00:34:30 2015 +0530
Merge branch 'master' into topic/master/net-virt-clustering
Conflicts:
features/ovsdb/src/main/features/features.xml
Change-Id: Ib614363bab8752913d430a86b420ab104010243c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
ddbb2eb2a22f271bf57e7470ef1420bd2301193d
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Thu Nov 19 04:50:03 2015 +0530
Switch to Alternate design of openflowplugin,
because alternate design supports clustering
Change-Id: Ia4e14bef316b2576298f013032232dc1e5d64984
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
13264eda1627d01fc6090008e326bcb91bb5e8a8
Merge:
99b5553 c66b0ff
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Tue Nov 17 01:05:38 2015 +0530
Merge branch 'master' into topic/master/net-virt-clustering
Conflicts:
openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/EgressAclServiceTest.java
openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtils.java
openstack/net-virt/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/impl/MdsalUtilsTest.java
Change-Id: I587fc39e1af228c8cff968ab2e5004285ae5bae2
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
99b5553d84f642d80890546643543a65a262645a
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Wed Oct 28 01:52:33 2015 +0530
Net-Virt Clustering : Enabled remote notifications
and modify the data store write methods to write only if
the instance is master instance
Change-Id: Icee31e0ea829f40bd64a072d840d03f8749c803f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
1e4144b81f47fad768141e7504ce7c2fb1e5f13f
Merge:
218e79b dc9f403
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Tue Oct 27 03:56:39 2015 +0530
Merge branch 'master' into topic/master/net-virt-clustering
Change-Id: I1cb9813a044d16dcfa152f6322271bcafe6bfd5c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
218e79b7bcf213ce64d2b55ff19e208015798aaf
Merge:
f91f1a3 6577da8
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Mon Oct 26 22:02:45 2015 +0530
Merge branch 'master' into topic/master/net-virt-clustering
Conflicts:
openstack/net-virt-providers/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/providers/NetvirtProvidersProvider.java
Change-Id: Ibc577f41a59c24ddc2b4cdb94b4a393f961e0950
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
f91f1a3883bf5a0d8a004e8d5141d2bdffd4aaa3
Merge:
fa8fe2a 342c8be
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Sat Oct 17 02:18:56 2015 +0530
Merge branch 'master' into topic/master/net-virt-clustering
Conflicts:
openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/EgressAclServiceTest.java
openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/IngressAclServiceTest.java
openstack/net-virt-sfc/features/pom.xml
openstack/net-virt-sfc/features/production/src/main/features/features.xml
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/NetvirtSfcProvider.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/AbstractTransactionCommandTest.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OpenVSwitchUpdateCommandTest.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbBridgeRemovedCommandTest.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbBridgeUpdateCommandTest.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbControllerRemovedCommandTest.java
Change-Id: I3cbe7ef4b0064bf07a489f2561630c4f669695fe
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
fa8fe2a789c741b2026e2946f08bf1e489db4794
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Thu Oct 8 16:17:01 2015 -0400
Wire EntityOwnership service into OVSDB NetVirt (part 2)
Change-Id: I222177c38ab8efeccd63924842d166633e88f606
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit
5239ecfcf1279e72d6d2446d4c88522461b28b1a)
commit
a93e284c51d476a1b3b8d4dd9ec532e0caa1c36f
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Thu Oct 8 13:35:40 2015 -0400
Wire EntityOwnership service into OVSDB NetVirt
Implemented logic for netvirt provider to participate in entityOwnership
service.
Change-Id: I8f014591099a952d49584eede07624f37492c27d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit
e43de9f997323cfe804857a0f68a01d95641fbc5)
commit
e74ec764cc75b295dc3e5babd36f44bb5d930b04
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Thu Oct 8 20:03:36 2015 +0530
Merge 'master' branch to 'topic/master/net-virt-clustering'
Squashed commit of the following:
commit
0cbdadc0af848566e34e66ae7a7ac952fd935561
Merge:
b02c878 cfcc6d4
Author: Sam Hague <shague@redhat.com>
Date: Wed Oct 7 22:42:02 2015 +0000
Merge "UT for southbound/transactions/md - set 2"
commit
b02c8783853d9308eb704ed7b2ca26d4d6d96933
Merge:
b063584 bf288f9
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Wed Oct 7 21:23:35 2015 +0000
Merge "If controller joins the cluster late, and the owner of the device is already decided, EntityOwnershipService won't notify new candidate that register for ownership. Adding a proactive check on the ownership state of the device, so if it's already owned, adding the connection instances in the connection cache."
commit
cfcc6d4b4288e44bdbcb9de4cf5ac0de97f741f6
Author: rpujar <rpujar@inocybe.com>
Date: Wed Oct 7 16:29:28 2015 -0400
UT for southbound/transactions/md - set 2
4. OvsdbBridgeUpdateCommand.java
5. OvsdbControllerRemovedCommand.java
Change-Id: Idc469c08351b1e98129dd9e798c07a660d782709
Signed-off-by: rpujar <rpujar@inocybe.com>
commit
b063584cc53ebf9c072bcb3171ef6d68bb727a8e
Author: Sam Hague <shague@redhat.com>
Date: Wed Oct 7 13:22:54 2015 -0400
Cleanup netvirtsfc poms
Change-Id: I6c1a3aa44c75f98d226dcf23dd514a31f5df5d2a
Signed-off-by: Sam Hague <shague@redhat.com>
commit
f5382918beb8096bfb68acf73084a3083c2e51d7
Author: Sam Hague <shague@redhat.com>
Date: Wed Oct 7 13:22:13 2015 -0400
Bump mdsal-utils version to match other bundles
Change-Id: I6c5166ae27fd2f055217be95630cbc482261bb61
Signed-off-by: Sam Hague <shague@redhat.com>
commit
7db67352c22b8275e81c0e919aa661eaa0caa1a3
Merge:
f183e98 ba1ac04
Author: Sam Hague <shague@redhat.com>
Date: Wed Oct 7 13:38:27 2015 +0000
Merge "UT for southbound/transactions/md"
commit
f183e9857b3a53645b2c463204a4f6945bb10cdb
Merge:
3a2524e fc26e9e
Author: Sam Hague <shague@redhat.com>
Date: Wed Oct 7 00:36:02 2015 +0000
Merge "Fix checkstyle issues in netvirtsfc"
commit
bf288f954947f35aca6425f7d180b4732dee6818
Author: Anil Vishnoi <vishnoianil@gmail.com>
Date: Wed Oct 7 02:11:25 2015 +0530
If controller joins the cluster late, and the owner of the device
is already decided, EntityOwnershipService won't notify new candidate that register
for ownership. Adding a proactive check on the ownership state of the device,
so if it's already owned, adding the connection instances in the connection cache.
Change-Id: I7e87b55599f0ebd408814d72789c86f231b7907c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
ba1ac0433a93468260f0e00e693ba1a928c14e14
Author: rpujar <rpujar@inocybe.com>
Date: Tue Oct 6 15:56:23 2015 -0400
UT for southbound/transactions/md
1. AbstractTransactionCommand.java
2. OpenVSwitchUpdateCommand.java
3. OvsdbBridgeRemovedCommand.java
Change-Id: I7655dec526bc06b770c533dd082eb403bc89a2c7
Signed-off-by: rpujar <rpujar@inocybe.com>
commit
3a2524e329b4955b96aa59eb8dbb8965d06f08a5
Merge:
99f9a37 e4783a5
Author: Sam Hague <shague@redhat.com>
Date: Tue Oct 6 19:48:18 2015 +0000
Merge "Send a canonical IPv4 prefix in table 60 routing rules"
commit
fc26e9e4667e470658ebcc9040f1274903075dcd
Author: Sam Hague <shague@redhat.com>
Date: Tue Oct 6 15:45:13 2015 -0400
Fix checkstyle issues in netvirtsfc
Change-Id: I7db518959449958849a74c49a83bde1135ee325f
Signed-off-by: Sam Hague <shague@redhat.com>
commit
e4783a5d2eca5670253b8251d62258edb01eb477
Author: Eric Multanen <eric.w.multanen@intel.com>
Date: Fri Oct 2 06:43:04 2015 -0700
Send a canonical IPv4 prefix in table 60 routing rules
Testing with userspace OVS with DPDK uncovered that the routing
rules that go into table 60 were getting rejected.
Messages from ovs-vswitchd.log:
... |nx_match|WARN|Rejecting NXM/OXM entry
0:32768:12:1:8 with 1-bits in value for bits wildcarded by the mask.
... |connmgr|INFO|br-int<->tcp:10.11.21.7:6653:
sending OFPBMC_BAD_WILDCARDS error reply to OFPT_FLOW_MOD message
Clearing the non-prefix bits resolved the problem and the flows were
successfully added.
v2 - split up long line into two more readable lines
Change-Id: I42a0c63e6e837197d29bb8333f12e822a5f15d91
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
commit
99f9a373e9bc410933421692148b87f177ab4879
Author: sangeeta.maurya <sangeeta.maurya@hp.com>
Date: Tue Oct 6 22:56:55 2015 +0545
Custom ICMP SG Rule:
1)Modification in ICMP matcher
2)Unit Test Improvement (Check added to validate if some values are set
properly).
Change-Id: I31fad80f6dd278372abe2b1a6aac95935aef5f17
Signed-off-by: sangeeta.maurya <sangeeta.maurya@hp.com>
Change-Id: I1b720089469f928ead3a317172901c6ce09ce1ef
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Change-Id: I2b79a3309cf29cf497edb3b8a4b5b5bcdaec9e18
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
commit
6f82804352d6373eb320c65ca0e777a7b18cd087
Author: karthik_j <karthik_j@hcl.com>
Date: Mon Jan 4 07:16:23 2016 +0530
Unit test file for NeutronFloatingIPInterface.
Change-Id: Ief42515748a32c2997bd7c6f43e5ae2b10321d50
Signed-off-by: karthik_j <karthik_j@hcl.com>
commit
32124667f903105c8d3a28ddb292dbe63ee03c30
Author: Sam Hague <shague@redhat.com>
Date: Wed Jan 6 21:04:20 2016 -0500
Disable hwvtep tests until impl is ready
Change-Id: I9bf643e55f59413ccb8220fdfc67c1b1b810a092
Signed-off-by: Sam Hague <shague@redhat.com>
commit
96062481ab888c718cd3bcebe3532bbef40b0f58
Author: Isaku Yamahata <isaku.yamahata@intel.com>
Date: Tue Jan 5 17:12:10 2016 -0800
Bug 4881 - failure in netvirt.AbstractEventTest
build fails with the following failure.
transaction id of the event can't be assumed to be 1 because other
tests may have created (subclass of) abstract event before running
this test. This patch to get transaction id from the event and use it
instead of hard-coded 1.
> Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.029 sec <<< FAILURE! - in org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest
> testAbstractEvent(org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest) Time elapsed: 0.017 sec <<< FAILURE!
> org.junit.ComparisonFailure: Error, toString() did not return the correct value expected:<...vent [transactionId=[1] handlerType=SOUTHBO...> but was:<...vent [transactionId=[5] handlerType=SOUTHBO...>
> at org.junit.Assert.assertEquals(Assert.java:115)
> at org.opendaylight.ovsdb.openstack.netvirt.AbstractEventTest.testAbstractEvent(AbstractEventTest.java:59)
Change-Id: I1f351b92a684e8ece17246d8f0011cbffe95409f
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
commit
7aead488ce87e8e7c41451ec3ba4e8408ce85e69
Author: Andre Fredette <afredette@redhat.com>
Date: Mon Jan 4 16:58:44 2016 -0500
Add netvirt and hwgw features to main ovsdb/features.
Also:
- Deleted ovsdb/netvirt/{artifacts,features,impl,karaf}.
- Update revision dates on yang files.
Change-Id: I61910ac934f806744bbb5c8d6c75daa8280d61f6
Signed-off-by: Andre Fredette <afredette@redhat.com>
commit
7f1836f062db0fdcd57a2641ade67776102f7f86
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Tue Jan 5 16:38:37 2016 +0100
Disable hwvtepsouthbound-it
Change-Id: I2a7537721240e8ef98c513771d0f7001f3206f31
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
e348249a3e098f6df48e1b68cd32dce75131bd44
Author: Stephen Kitt <skitt@redhat.com>
Date: Tue Jan 5 11:00:52 2016 +0100
Remove unused SchemaVersionMismatchException constructors
Change-Id: I9ad97e6332b2db75ae080a09aa4d9199ba89adb2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
d5bd15d694a6c8abd3cb1d41bc4c5ee1c68ae7fc
Author: Stephen Kitt <skitt@redhat.com>
Date: Tue Jan 5 10:59:54 2016 +0100
Add a unit test for TyperUtils::checkVersion()
Change-Id: Ibb133dfc0a225c275a9cd6cbdb1fb060a8e97e17
Signed-off-by: Stephen Kitt <skitt@redhat.com>
commit
fe755734e89e28ebe22ed5ad783d303e7d4ae08e
Author: Eric Multanen <eric.w.multanen@intel.com>
Date: Tue Dec 15 14:32:11 2015 -0800
QoS Schema - fix type of QoS type
It looks like the 'type' field in the QoS table schema
was incorrectly coded in the Qos.java schema file
to be a Set<String>.
In the ovs schema, the type field for Qos is defined
as a simple string:
"type": {
"type": "string"},
Other string fields in the ovs schema that are
defined in the <table>.java files as Set<String>
appear in the schema as follows (for example,
ovs_version from Open_vSwitch table):
"ovs_version": {
"type": {"key": {"type": "string"},
"min": 0, "max": 1}},
With the QoS 'type' field defined as Set<String>,
the code gets an exception trying to cast
a String to a Set<String> when trying to get
the type out of a table update. After this patch
is applied, the error is resolved.
Change-Id: Ibb42db03e71d8f1d96329b5942e7501069af20cb
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
commit
dfc18e8b3a851ffc2b10518ec59fa97e5c153c52
Author: Flavio Fernandes <ffernand@redhat.com>
Date: Mon Jan 4 13:52:05 2016 -0500
Bug 4874 - distributed arp in old l3 for ovsdb is not installing rules when it should
Fix TenantNetworkManagerImpl::isTenantNetworkPresentInNode(), so it does not
use stale node to determine the termination points.
Patch set 2: fix UT
Change-Id: I22e1e88ca5b37f6da28b2281f01aaec07c3f751e
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
commit
99ec887f6397c622aca327208ced9630dabdb97d
Author: Rashmi Pujar <rpujar@inocybe.com>
Date: Mon Jan 4 13:31:50 2016 -0500
Fix UT: call method under test and verify
Change-Id: I7f4dedfb761b23abc04909e5ed6a5a9330c9ee72
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
commit
9484b0d2721fdd95b9dea97f02db47dcf7580fd5
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Thu Dec 24 14:28:59 2015 +0530
Added more ITs to hwvtep
Adds following tests and required utils code:
1. testHwvtepNodeConnectDisconnect
2. testAddDeletePhysicalSwitch
Fixes issues found while writing and running ITs
Change-Id: Ia1b4bb03e228dd16c7b01dedc0ef1a4b3d3ea256
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
9b3005bd8034e9ef2dec29567094e2891cd6d818
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Sun Dec 27 19:13:58 2015 +0530
Add Tunnel configuration
Yang changes to configure tunnels [including bfd]
Code to store tunnels information in Operational DataStore.
Code to configure tunnels
Change-Id: Ibd882c6737305af9931488e4cda394e92bcdaf91
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
eced0bdd8ec2d9f52928d7d0ef6de649b3865270
Merge:
42cd637 094b3e9
Author: Sam Hague <shague@redhat.com>
Date: Thu Dec 24 17:48:15 2015 +0000
Merge "Added support for port range in TCP and UDP security group rules"
commit
094b3e92ef3d4c7deaefa4cbb1eeabe6fb89c627
Author: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
Date: Fri Dec 25 06:07:15 2015 +0530
Added support for port range in TCP and UDP security group rules
Change-Id: I69ecc8eb52bedb641a7b2c4407b3e070ed2c1677
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>
commit
42cd6371da33c0fdd5ba26e2d84915836abd4562
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Thu Dec 24 15:23:05 2015 +0530
Fix hwvtep.yang
BFD configuration and parameters are defined as string. This is incorrect
and makes it difficult to use BFD cofigs and params. This fix creates
them as lists of key:value pairs of type string:string.
Change-Id: I8af37e443cc04d3665ef4c70824b02d71c14b9b5
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
1720eb6d6809586c2cd1cac75e15825aceeff2fe
Merge:
60048d9 ad34a3d
Author: Sam Hague <shague@redhat.com>
Date: Wed Dec 23 23:37:30 2015 +0000
Merge "First netvirt renderer framework: Hardware Gateway (hwgw)"
commit
60048d9526858b887eeceb822f29c5e5408796d3
Merge:
4a22575 fb2f6a3
Author: Sam Hague <shague@redhat.com>
Date: Wed Dec 23 23:29:43 2015 +0000
Merge "Fix NPE in VLAN Bindings deletion"
commit
4a225754c31f397a1255d945256412305e8e9044
Author: Vishal Thapar <vishal.thapar@ericsson.com>
Date: Fri Dec 18 11:30:07 2015 +0530
ITs for hwvtepsouthbound
Enables IT for hwvtepsouthbound
adds hwvtepsouthboundutils to utils
Change-Id: I365604e75aa45c9a67312b2b275b4041141d75e1
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
commit
fb2f6a3dc732a905147091b3b1e292cef1e102d7
Author: karanrajsingh <karan.raj.s.singh@ericsson.com>
Date: Wed Dec 23 15:46:13 2015 +0530
Fix NPE in VLAN Bindings deletion
Fixed NPE for multiple physical port addition.
Change-Id: I4b7324bd00dc086d843bb9f1327683dcac696e3d
Signed-off-by: karanrajsingh <karan.raj.s.singh@ericsson.com>
commit
ad34a3d8d08a0742a75329c3db9c7ab72c8dbe66
Author: Andre Fredette <afredette@redhat.com>
Date: Tue Dec 22 18:52:30 2015 -0500
First netvirt renderer framework: Hardware Gateway (hwgw)
Change-Id: Ic84ba7c7fda140e1bba304541da2d7cc4c0254de
Signed-off-by: Andre Fredette <afredette@redhat.com>
Signed-off-by: Mohnish Anumala <Mohnish_Anumala@Dell.com>