cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chip Childers <chip.child...@sungard.com>
Subject Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM
Date Fri, 22 Mar 2013 15:31:09 GMT
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-dev@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:noreply@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
> 

Mime
View raw message