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: additional patch for CLOUDSTACK-685
Date Wed, 15 May 2013 14:39:08 GMT
Should this be applied to 4.1?

-chip

On Wed, May 15, 2013 at 11:16:57AM +0000, Kishan Kavala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11157/#review20572
> -----------------------------------------------------------
> 
> Ship it!
> 
> 
> commit 737063891592af1d896f150cca02ff3d63a0198c
> 
> - Kishan Kavala
> 
> 
> On May 14, 2013, 6:02 p.m., Wei Zhou wrote:
> > 
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/11157/
> > -----------------------------------------------------------
> > 
> > (Updated May 14, 2013, 6:02 p.m.)
> > 
> > 
> > Review request for cloudstack and Kishan Kavala.
> > 
> > 
> > Description
> > -------
> > 
> > This is the additional patch for CLOUDSTACK-685 (and CLOUDSTACK-721 as well) after
this commit
> > https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=commit;h=cd7f7716155c3cd2d29b97d7ea302518d81fdfab
> > 
> > PrepareStop part in VirtualNetworkApplianceImpl.java need to be changed as well.
> > 
> > 
> > This addresses bug CLOUDSTACK-685.
> > 
> > 
> > Diffs
> > -----
> > 
> >   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java f6057e8

> > 
> > Diff: https://reviews.apache.org/r/11157/diff/
> > 
> > 
> > Testing
> > -------
> > 
> > Testing ok.
> > 
> > 
> > Thanks,
> > 
> > Wei Zhou
> > 
> >
> 

Mime
View raw message