> On June 10, 2014, 1:49 p.m., daan Hoogland wrote: > > systemvm/patches/debian/config/etc/init.d/cloud-early-config, line 39 > > <https://reviews.apache.org/r/21771/diff/1/?file=586596#file586596line39> > > > > wouldn't 2>/dev/null be more appropriate for the described objective?
after off-line discussion with Joris I will apply - daan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45229 ----------------------------------------------------------- On May 21, 2014, 3:05 p.m., Joris van Lieshout wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21771/ > ----------------------------------------------------------- > > (Updated May 21, 2014, 3:05 p.m.) > > > Review request for cloudstack, Chiradeep Vittal, daan Hoogland, Rajesh > Battala, Hugo Trippaers, and Sheng Yang. > > > Repository: cloudstack-git > > > Description > ------- > > 1. On start hv_kvp_daemon if it exists. Currently trows an error on the > console during boot. > 2. By adding -f to the rm of boot_up_done it will be silent if it does not > exist. Currently also trows an error during bootup. > 3. Use log_action_msg instead of log_action_begin_msg will make the console > look cleaner. > > > Diffs > ----- > > systemvm/patches/debian/config/etc/init.d/cloud-early-config ffb23ec > > Diff: https://reviews.apache.org/r/21771/diff/ > > > Testing > ------- > > We are running these fixes in our beta and prod env for a while allready > > > Thanks, > > Joris van Lieshout > >
