----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9908/#review18244 -----------------------------------------------------------
Commit 0d604c636daed253f8d3034d8093e56f1ab85aff in branch refs/heads/master from Prasanna Santhanam <t...@apache.org> [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=0d604c6 ] 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 > >
