> On April 18, 2013, 11:36 p.m., Chiradeep Vittal wrote: > > patches/systemvm/debian/config/root/userdata.sh, line 79 > > <https://reviews.apache.org/r/10614/diff/1/?file=282479#file282479line79> > > > > why is this needed? shouldn't the first rewrite rule be sufficient? > >
See comments above. - ilya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10614/#review19416 ----------------------------------------------------------- On April 18, 2013, 11:02 p.m., ilya musayev wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10614/ > ----------------------------------------------------------- > > (Updated April 18, 2013, 11:02 p.m.) > > > Review request for cloudstack and Chiradeep Vittal. > > > Description > ------- > > Addressed 2 issues: > Rewrote the bash if/else check to fix redundant entries > Added RewriteRules to support boto/cloud-init - as per Chiradeep - this would > only affect VMWare deployments, as this userdata.sh script is only used with > vmware system router vms > > > This addresses bug CLOUDSTACK-2053. > > > Diffs > ----- > > patches/systemvm/debian/config/root/userdata.sh e5d170b > > Diff: https://reviews.apache.org/r/10614/diff/ > > > Testing > ------- > > ACS4.1 > > > Thanks, > > ilya musayev > >
