Actually I just applied it. So it's okay. On Wed, Jul 24, 2013 at 05:34:27PM +0000, Sanjay Tripathi wrote: > Can you rebase your branch and try to create a new patch; file that you > modified might have got updated. > > --Sanjay > > > -----Original Message----- > > From: Sowmya Krishnan [mailto:sowmya.krish...@citrix.com] > > Sent: Wednesday, July 24, 2013 10:37 PM > > To: dev@cloudstack.apache.org; Prasanna Santhanam > > Subject: RE: Review Request 12874: Fix reboot failures with wait time > > > > RB doesn???t accept the patch. Keeps failing. Here's the patch: > > > > From 02238e6903d3d399d402886513efd6cd2664696a Mon Sep 17 00:00:00 > > 2001 > > From: Sowmya Krishnan <sowmya.krish...@citrix.com> > > Date: Tue, 23 Jul 2013 22:29:10 +0530 > > Subject: [PATCH] Add wait while Netscaler recognizes service to be up > > > > --- > > test/integration/component/test_netscaler_lb.py | 2 ++ > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > diff --git a/test/integration/component/test_netscaler_lb.py > > b/test/integration/component/test_netscaler_lb.py > > index cd85659..d93a262 100644 > > --- a/test/integration/component/test_netscaler_lb.py > > +++ b/test/integration/component/test_netscaler_lb.py > > @@ -2772,6 +2772,8 @@ class TestVmWithLb(cloudstackTestCase): > > self.debug("Rebooting VM instance: %s" % self.vm_2.name) > > self.vm_2.reboot(self.apiclient) > > self.debug("Rebooting VM: %s" % self.vm_2.name) > > + self.debug("Waiting for Netscaler to get the service up") > > + time.sleep( 120 ) > > > > try: > > self.debug( > > -- > > 1.7.5.4 > > > > > > > -----Original Message----- > > > From: Prasanna Santhanam [mailto:nore...@reviews.apache.org] On > > Behalf Of > > > Prasanna Santhanam > > > Sent: Wednesday, July 24, 2013 9:56 PM > > > To: Prasanna Santhanam > > > Cc: cloudstack; Sowmya Krishnan > > > Subject: Re: Review Request 12874: Fix reboot failures with wait time > > > > > > > > > ----------------------------------------------------------- > > > This is an automatically generated e-mail. To reply, visit: > > > https://reviews.apache.org/r/12874/#review23766 > > > ----------------------------------------------------------- > > > > > > > > > I think you forgot to attach the diff here. > > > > > > - Prasanna Santhanam > > > > > > > > > On July 23, 2013, 5:20 p.m., Sowmya Krishnan wrote: > > > > > > > > ----------------------------------------------------------- > > > > This is an automatically generated e-mail. To reply, visit: > > > > https://reviews.apache.org/r/12874/ > > > > ----------------------------------------------------------- > > > > > > > > (Updated July 23, 2013, 5:20 p.m.) > > > > > > > > > > > > Review request for cloudstack and Prasanna Santhanam. > > > > > > > > > > > > Repository: cloudstack-git > > > > > > > > > > > > Description > > > > ------- > > > > > > > > Fix reboot failures with sleep time since Netscaler at times takes > > > > almost > > 60 - > > > 90 seconds for identifying the service to be up after rebooting a VM. > > > > > > > > > > > > Diffs > > > > ----- > > > > > > > > > > > > Diff: https://reviews.apache.org/r/12874/diff/ > > > > > > > > > > > > Testing > > > > ------- > > > > > > > > Tested locally and the script now passes > > > > > > > > > > > > Thanks, > > > > > > > > Sowmya Krishnan > > > > > > > > >
-- Prasanna., ------------------------ Powered by BigRock.com