----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13759/ -----------------------------------------------------------
(Updated Aug. 28, 2013, 4:35 a.m.) Review request for cloudstack and Sheng Yang. Changes ------- Rebased patch against master so that it can be applied there. Bugs: CLOUDSTACK-4466 Repository: cloudstack-git Description ------- Changes in this patch are entirely confined to the MidoNet plugin itself. A recent code change in NetworkManager causes NullPointerExceptions when DHCP capability list is null. The commit which made the NetworkManager change also changed the VirtualRouter to not use null for the capabilitylist, but didn't make this change for other network devices, causing DHCP to fail on MidoNet. This change also updates the MidoNet plugin to use the most recent MidoNet API. Diffs (updated) ----- plugins/network-elements/midonet/pom.xml 9d6588b plugins/network-elements/midonet/src/com/cloud/network/element/MidoNetElement.java 85814ef plugins/network-elements/midonet/src/com/cloud/network/element/SimpleFirewallRule.java a6b78d8 plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetGuestNetworkGuru.java ac77cf6 plugins/network-elements/midonet/src/com/cloud/network/resource/MidoNetVifDriver.java 3c7c23d plugins/network-elements/midonet/test/com/cloud/network/element/MidoNetElementTest.java 9023c6e Diff: https://reviews.apache.org/r/13759/diff/ Testing ------- Removed Maven cache (rm -rf ~/.m2/repository) and built successfully, tests passed. Spun up several networks and VMs, tested ping to outside, ping VM to VM, DHCP, Source NAT, Static NAT, Port Forwarding. Thanks, Dave Cahill