Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-05-21 Thread Joris van Lieshout
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/ --- Review request for cloudstack, Chiradeep Vittal, daan Hoogland, Rajesh Battala,

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45229 --- systemvm/patches/debian/config/etc/init.d/cloud-early-config

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-12 Thread daan Hoogland
> On June 10, 2014, 1:49 p.m., daan Hoogland wrote: > > systemvm/patches/debian/config/etc/init.d/cloud-early-config, line 39 > > > > > > wouldn't 2>/dev/null be more appropriate for the described objective? after off-

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-12 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45492 --- Ship it! 66816827cbef14153f734e494c8adc41fbae - daan Hoogland

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-12 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45493 --- systemvm/patches/debian/config/etc/init.d/cloud-early-config

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-17 Thread Joris van Lieshout
> On June 12, 2014, 1:11 p.m., Rajesh Battala wrote: > > systemvm/patches/debian/config/etc/init.d/cloud-early-config, line 33 > > > > > > This daemon will be installed in the base image itself. Hi Rajesh, I don't know