Re: Review Request 23168: Update cloudstack-usage JAVA_HOME to find JRE for Java 7

[Sebastien Goasguen <ru...@gmail.com>]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23168/#review47298
-----------------------------------------------------------


Ok, there was actually an earlier patch https://reviews.apache.org/r/22721/
Which I think was a bit nicer, so I used that one instead.
It has been committed to 4.3.1

thanks, can you close the review?

- 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
> 
>