Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

Rajani, I had missed this. My apologies. The status shows as submitted. I am guessing that you have updated the status and nothing more needs to be done. Saurav On Mon, Jul 28, 2014 at 11:48 AM, Rajani Karuturi wrote: >This is an automatically generated e-mail. To reply, visit: > https://

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

> On April 29, 2014, 12:29 p.m., Santhosh Edukulla wrote: > > Ship It! > > Santhosh Edukulla wrote: > If we don't have any review issues pending, can some body with commit > rights push this patch? > > Santhosh Edukulla wrote: > Saurav, > > Please update BugId and Branch field

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

> On April 29, 2014, 12:29 p.m., Santhosh Edukulla wrote: > > Ship It! > > Santhosh Edukulla wrote: > If we don't have any review issues pending, can some body with commit > rights push this patch? Saurav, Please update BugId and Branch fields. This review submission is pending since lon

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

Hi Saurav, Can you please file bug describing the issue ? Also please update the bug-id in review request. Thanks, Jayapal On 24-May-2014, at 12:24 AM, Santhosh Edukulla wrote: > > >> On April 29, 2014, 12:29 p.m., Santhosh Edukulla wrote: >>> Ship It! > > If we don't have any review issue

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

> On April 29, 2014, 12:29 p.m., Santhosh Edukulla wrote: > > Ship It! If we don't have any review issues pending, can some body with commit rights push this patch? - Santhosh --- This is an automatically generated e-mail. To reply, v

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/#review41708 --- Ship it! Ship It! - Santhosh Edukulla On April 23, 2014, 9:20 a.

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

> On April 25, 2014, 10:10 a.m., Santhosh Edukulla wrote: > > systemvm/scripts/run.sh, line 44 > > > > > > Just to know, is _run.sh called only from here or has other references > > as well? If yes, then this logic doe

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/#review41457 --- systemvm/scripts/run.sh

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/ --- (Updated April 23, 2014, 9:20 a.m.) Review request for cloudstack, Jayapal Redd

RE: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes Santosh, Can you let me know ur thoughs If the review comments have been adequately addressed and the changes looks ok? Thanks Saurav On Tue, Apr 15, 2014 at 5:43 PM, Saurav

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

Santosh, Can you let me know ur thoughs If the review comments have been adequately addressed and the changes looks ok? Thanks Saurav On Tue, Apr 15, 2014 at 5:43 PM, Saurav Lahiri wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20123/ >

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/ --- (Updated April 15, 2014, 12:13 p.m.) Review request for cloudstack, Jayapal Red

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

> On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote: > > systemvm/scripts/utils.sh, line 9 > > > > > > Where is the variable CLOUD_COM_HOME defined? This is a typo. I have fixed this with the new patch file. > On

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/ --- (Updated April 15, 2014, 11:50 a.m.) Review request for cloudstack, Jayapal Red

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/#review40251 --- systemvm/scripts/run.sh

Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/ --- (Updated April 14, 2014, 7:15 a.m.) Review request for cloudstack, Jayapal Redd

Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20123/ --- Review request for cloudstack, Rajani Karuturi and Rajesh Battala. Repository:

Re: Prevent console proxy support scripts from spawning multiple java processes

Raised defect : https://issues.apache.org/jira/browse/CLOUDSTACK-6328 Thanks Saurav On Tue, Apr 1, 2014 at 9:06 PM, Saurav Lahiri wrote: > Hi, > > The console proxy supporting scripts run.sh restarts the java process if > it determines that > for some reason the _run.sh has exited. If the origi

Prevent console proxy support scripts from spawning multiple java processes

Hi, The console proxy supporting scripts run.sh restarts the java process if it determines that for some reason the _run.sh has exited. If the original java process has exited then that is not an issue. If for some reason the java process is still running, then the while loop which calls _run.sh w