----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23168/#review47000 -----------------------------------------------------------
After the updated review it looks like you are only changing the centOS scripts. What about Ubuntu/debs ? is that already fixed ? - Sebastien Goasguen On June 30, 2014, 3:54 p.m., Olivier Lemasle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23168/ > ----------------------------------------------------------- > > (Updated June 30, 2014, 3:54 p.m.) > > > Review request for cloudstack and Sebastien Goasguen. > > > Repository: cloudstack-git > > > Description > ------- > > cloudstack-usage is currently configured to search JAVA_HOME only in a small > set of paths corresponding to Java 6 (/usr/lib/jvm/java-6-openjdk, > /usr/lib/jvm/java-6-openjdk-i386, etc). > > If cloudstack-usage is installed on a server with only Java 7, the service > cannot start and fails with the following error: > jsvc error: Cannot locate Java Home > > This commit adds Java 7 paths to the possible set of JAVA_HOME locations. > > > Diffs > ----- > > packaging/centos63/cloud-usage.rc e78f8c1 > > Diff: https://reviews.apache.org/r/23168/diff/ > > > Testing > ------- > > Package CloudStack RPMs for CentOS and install them on a system with > java-1.7.0-openjdk and no Java 6 JRE. > Check that cloudstack-usage works correctly. > > > Thanks, > > Olivier Lemasle > >
