please go ahead, I will change BridgeVifDriver as per https://reviews.apache.org/r/13934. i see no conflicts
On Wed, Sep 18, 2013 at 3:48 PM, Wei Zhou <w.z...@leaseweb.com> wrote: > > >> On Sept. 9, 2013, 2:34 p.m., Wei Zhou wrote: >> > please ignore the r2 version for issue CLOUDSTACK-3565 > > CLOUDSTACK-3565 is fixed, so I will commit r2 version if nobody objects. > > > - Wei > > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13992/#review25989 > ----------------------------------------------------------- > > > On Sept. 9, 2013, 8:09 a.m., Wei Zhou wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/13992/ >> ----------------------------------------------------------- >> >> (Updated Sept. 9, 2013, 8:09 a.m.) >> >> >> Review request for cloudstack and edison su. >> >> >> Bugs: CLOUDSTACK-4405 >> >> >> Repository: cloudstack-git >> >> >> Description >> ------- >> >> There still exist two issues after Edison's commits. >> (1) Migration from new hosts to old hosts failed. >> The bridge name on old host is set to cloudVirBr* if >> network.bridge.name.schema is set to 3.0 in >> /etc/cloudstack/agent/agent.properties, but the actual bridge name is >> breth*-* after running cloudstack-agent-upgrade. >> >> (2) all ports of vms (Basic zone, or Advanced zone with security groups) on >> old hosts are open, because the iptables rules are binding to device >> (bridge) name which is changed by cloudstack-agent-upgrade. >> >> >> Diffs >> ----- >> >> agent/bindir/cloudstack-agent-upgrade.in 4972d39 >> debian/cloudstack-agent.postinst 499ae6a >> debian/rules 5e3d58c >> packaging/centos63/cloud.spec 2b814f8 >> >> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java >> e3779a7 >> scripts/vm/network/security_group.py 0ac8b74 >> >> Diff: https://reviews.apache.org/r/13992/diff/ >> >> >> Testing >> ------- >> >> tested ok on my environment. >> >> After this, the KVM upgrade steps : >> a. Install 4.2 cloudstack agent on each kvm host >> b. Run "cloudstack-agent-upgrade". This script will upgrade all the existing >> bridge name to new bridge name, and update related firewall rules. >> c. install a libvirt hook: >> c1. mkdir /etc/libvirt/hooks >> c2. cp /usr/share/cloudstack-agent/lib/libvirtqemuhook >> /etc/libvirt/hooks/qemu >> c3. chmod +x /etc/libvirt/hooks/qemu >> c4. service libvirtd restart >> c5. service cloudstack-agent restart >> >> >> Thanks, >> >> Wei Zhou >> >> >