----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7187/ -----------------------------------------------------------
(Updated Sept. 25, 2012, 11:44 p.m.) Review request for cloudstack and edison su. Changes ------- checks to see if old style cloudVirBr names are in use, and uses those bridges if found. Description ------- Cloudstack seems to let you create guest traffic types on multiple physical networks. However, when I try this with KVM I end up always bridging to whatever device is used for guest.network.device. This pulls the traffic label (NicTO.getName()) and uses that bridge to ensure that we get on the correct physical network, rather than just always using the guest.network.device. This also changes the bridge naming scheme from cloudVirBr + vlanid to br + physicalinterface + "-" + vlanid. This is because we should be able to support the same vlan numbers per physical network, and the previous bridge name would not support this and collide. Diffs (updated) ----- plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java cf4de09 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java aadd6dd scripts/vm/network/vnet/modifyvlan.sh 33d697a Diff: https://reviews.apache.org/r/7187/diff/ Testing ------- Ran this in our test environment, which is running the latest 4.0 + this patch. Everything operates as expected, but since I don't have the environment or resources to test this in every situation I'd hope someone could give this a detailed look over. Regardless, it's a fairly straightforward fix. Thanks, Marcus Sorensen
