> On June 12, 2014, 1:11 p.m., Rajesh Battala wrote: > > systemvm/patches/debian/config/etc/init.d/cloud-early-config, line 33 > > <https://reviews.apache.org/r/21771/diff/1/?file=586596#file586596line33> > > > > This daemon will be installed in the base image itself.
Hi Rajesh, I don't know what you mean by this comment? This daemon is for hyperV only, correct? So, on other hypervisors it does not exist making this a nice way to suppress the message that the script cannot be found, right? - Joris ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45493 ----------------------------------------------------------- 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 > >