Re: Review Request: additional patch for CLOUDSTACK-685
--- 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 > >
Re: Review Request: additional patch for CLOUDSTACK-685
Should this be applied to 4.1? -chip On Wed, May 15, 2013 at 11:16:57AM +, 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 > > > > >
Re: Review Request: additional patch for CLOUDSTACK-685
Hi Chip, Yes. and the patch for CLOUDSTACK-962 as well. Thanks! https://reviews.apache.org/r/11181/ Wei 2013/5/15 Chip Childers > Should this be applied to 4.1? > > -chip > > On Wed, May 15, 2013 at 11:16:57AM +, 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 > > > > > > > > >
Re: Review Request: additional patch for CLOUDSTACK-685
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11157/#review20587 --- Ship it! Committed to 4.1: commit aed9a623d81c922af53e9c1f7116c1c7c85fdda5 Author: Wei Zhou Date: Wed May 15 15:54:51 2013 +0100 CLOUDSTACK-685: 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. Signed-off-by: Chip Childers - Chip Childers 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 > >
Re: Review Request: additional patch for CLOUDSTACK-685
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11157/#review20588 --- Commit aed9a623d81c922af53e9c1f7116c1c7c85fdda5 in branch refs/heads/4.1 from Chip Childers [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=aed9a62 ] CLOUDSTACK-685: 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. Signed-off-by: Chip Childers - ASF Subversion and Git Services 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 > >
Re: Review Request: additional patch for CLOUDSTACK-685
On Wed, May 15, 2013 at 04:48:58PM +0200, Wei ZHOU wrote: > Hi Chip, > > Yes. and the patch for CLOUDSTACK-962 as well. Thanks! > https://reviews.apache.org/r/11181/ > > Wei Both were applied to 4.1. Can you please review to be sure that they are in correctly. Also, do either patch still need to be applied to master? Specify the review or commit sh and I'll push it over there. > > > 2013/5/15 Chip Childers > > > Should this be applied to 4.1? > > > > -chip > > > > On Wed, May 15, 2013 at 11:16:57AM +, 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 > > > > > > > > > > > > >