----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10104/ -----------------------------------------------------------
(Updated March 25, 2013, 5:16 p.m.) Review request for cloudstack. Changes ------- Updated patch to include some things I missed. Description ------- Update for CLOUDSTACK-1736. The newly created package for cloudstack-management was not correctly installing the service. This prevented cloud-setup-management from being able to configure the service, and the init script didn't even believe the service was installed. I also added sudo to the chmod command for checking script permissions, as most scripts belong to root. This addresses bug CLOUDSTACK-1736. Diffs (updated) ----- debian/cloudstack-agent.install b142571 debian/cloudstack-management.install cecc311 debian/cloudstack-management.postinst 7b6a1ed debian/rules 613d76a packaging/debian/init/cloud-management 490bf1e packaging/debian/replace.properties fee1def server/src/com/cloud/server/ConfigurationServerImpl.java 3b0ec0f Diff: https://reviews.apache.org/r/10104/diff/ Testing ------- Did a full compile and build of all debs. Installed debs on an Ubuntu 12.04 box, ran cloud-setup-management, and was able to successfully login to the gui. Thanks, Brian Spindler