----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7194/#review11741 -----------------------------------------------------------
Ship it! Ship It! - edison su On Sept. 20, 2012, 5:55 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7194/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2012, 5:55 p.m.) > > > Review request for cloudstack and edison su. > > > Description > ------- > > Recent change was made to ignore the systemvm.iso in KVM and only mount the > root and patch disks. As a result, the /etc/init.d/cloud-early-config script > no longer attempts to copy the authorized_keys file, since the copy command > is in an if statement that looks for the cdrom. This patch copies the > authorized_keys for kvm. > > Update: the system vm iso is the bigger issue, and I'm submitting a separate > fix for that, but I think it makes sense to not depend on the cdrom being > mounted to copy the /var/cache authorized keys since it's a function of the > patch disk, so this patch still applies. > > > Diffs > ----- > > patches/systemvm/debian/config/etc/init.d/cloud-early-config ca97d29 > > Diff: https://reviews.apache.org/r/7194/diff/ > > > Testing > ------- > > tested against latest 4.0 > > > Thanks, > > Marcus Sorensen > >
