----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9908/#review18280 -----------------------------------------------------------
Commit f4f78311b3b5ef5c404a4d06b6832cb3b1c2c4c4 in branch refs/heads/4.1 from Chip Childers <chip.child...@gmail.com> [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=f4f7831 ] CLOUDSTACK-1666: Fix NetworkUsage for VPC on KVM The collection of network usage from VPC virtual router on KVM does not work, because there is no corresponding procedure to deal with VPC virtual router (cmd.isForVpc() = true). Reviewed-by: Kishan Kavala <kis...@apache.org> Reported-by: Wei Zhou <w.z...@leaseweb.com> Signed-off-by: Prasanna Santhanam <t...@apache.org> - ASF Subversion and Git Services On March 19, 2013, 2:26 p.m., Wei Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9908/ > ----------------------------------------------------------- > > (Updated March 19, 2013, 2:26 p.m.) > > > Review request for cloudstack, Kishan Kavala and edison su. > > > Description > ------- > > add procedure VPCNetworkUsage and getVPCNetworkStats to deal with the > operation on VPC virtual router. > "create" to add iptables rules for network usage statistics. > "get" to get the network usage statistics from iptables. > > > This issue affects 4.0.1/4.1/master. However, this patch only applies on > 4.0.1. > > > This addresses bug CLOUDSTACK-1666. > > > Diffs > ----- > > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > 8ee3ea4 > > Diff: https://reviews.apache.org/r/9908/diff/ > > > Testing > ------- > > Testing manually ok. > > > Thanks, > > Wei Zhou > >
