Think Wei already provided that. I applied it to master this morning. But it 
will be reqd for 4.1 as well.


~P.

-----Original Message-----
From: Chip Childers <chip.child...@sungard.com>
Date: Fri, 22 Mar 2013 11:31:09 
To: Kishan Kavala<kishan.kav...@citrix.com>
Reply-To: dev@cloudstack.apache.org
Cc: Animesh Chaturvedi<animesh.chaturv...@citrix.com>; 
cloudstack-...@incubator.apache.org<cloudstack-...@incubator.apache.org>; Wei 
Zhou<w.z...@leaseweb.com>; Edison Su<edison...@citrix.com>
Subject: Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM

On Fri, Mar 22, 2013 at 12:56:27PM +0000, Kishan Kavala wrote:
> Chip,
>  Can please you apply this patch to 4.1. It is pushed to 4.0 and master 
> already.

Cherry picking this from 4.0 to master fails.  We need a clean patch for
those branches please!

> 
> Thanks,
> kishan
> 
> > -----Original Message-----
> > From: Animesh Chaturvedi
> > Sent: Friday, 15 March 2013 12:41 AM
> > To: cloudstack-...@incubator.apache.org; Wei Zhou; Kishan Kavala; Edison
> > Su
> > Subject: RE: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC
> > on KVM
> > 
> > Wei
> > 
> > Please add Kishan and Edison as reviewers. Folks please review and apply
> > patch
> > 
> > Animes
> > 
> > > -----Original Message-----
> > > From: Wei Zhou [mailto:nore...@reviews.apache.org] On Behalf Of Wei
> > > Zhou
> > > Sent: Wednesday, March 13, 2013 12:21 PM
> > > To: cloudstack; Wei Zhou
> > > Subject: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC
> > on
> > > KVM
> > >
> > >
> > > -----------------------------------------------------------
> > > This is an automatically generated e-mail. To reply, visit:
> > > https://reviews.apache.org/r/9908/
> > > -----------------------------------------------------------
> > >
> > > Review request for cloudstack.
> > >
> > >
> > > 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 addresses bug CLOUDSTACK-1666.
> > >
> > >
> > > Diffs
> > > -----
> > >
> > >
> > >
> > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtC
> > > o
> > > mputingResource.java b8cce1e
> > >
> > > Diff: https://reviews.apache.org/r/9908/diff/
> > >
> > >
> > > Testing
> > > -------
> > >
> > > Testing manually ok.
> > >
> > >
> > > Thanks,
> > >
> > > Wei Zhou
> 

Reply via email to