No, only need for master. I just checked 4.1 and it does not have the changes from commit 5cd3608.
On Mon, Apr 15, 2013 at 1:07 PM, Chip Childers <chip.child...@sungard.com>wrote: > Do I need a patch for this in 4.1? > > On Mon, Apr 15, 2013 at 05:02:07PM +0000, Chiradeep Vittal wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/10437/#review19193 > > ----------------------------------------------------------- > > > > Ship it! > > > > > > Ship It! > > > > - Chiradeep Vittal > > > > > > On April 12, 2013, 9:11 p.m., Phong Nguyen wrote: > > > > > > ----------------------------------------------------------- > > > This is an automatically generated e-mail. To reply, visit: > > > https://reviews.apache.org/r/10437/ > > > ----------------------------------------------------------- > > > > > > (Updated April 12, 2013, 9:11 p.m.) > > > > > > > > > Review request for cloudstack and Hugo Trippaers. > > > > > > > > > Description > > > ------- > > > > > > Management server unable to start (from rpm) due to invalid call to > Script for injectkeys.sh. > > > > > > 2013-04-12 15:02:13,496 WARN [cloud.server.ConfigurationServerImpl] > (Timer-1:null) Failed to inject generated public key into systemvm iso > java.io.IOException: Cannot run program "/bin/bash > /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2, > No such file or directory > > > at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029) > > > at com.cloud.utils.script.Script.execute(Script.java:183) > > > > > > > > > Diffs > > > ----- > > > > > > server/src/com/cloud/server/ConfigurationServerImpl.java 596387f > > > > > > Diff: https://reviews.apache.org/r/10437/diff/ > > > > > > > > > Testing > > > ------- > > > > > > Rebuilt rpm and tested management server startup -- success > > > > > > > > > Thanks, > > > > > > Phong Nguyen > > > > > > > > >
