[
https://issues.apache.org/jira/browse/CLOUDSTACK-7476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14122950#comment-14122950
]
ASF GitHub Bot commented on CLOUDSTACK-7476:
--------------------------------------------
Github user lsimons commented on the pull request:
https://github.com/apache/cloudstack/pull/15#issuecomment-54626548
Sweet, thanks!
Personally I think just having it on master is ok -- anyone that is likely
to run into this issue is also likely to have configuration management (like
chef) in place to edit their cloudstack-usage script if they need to do so.
Of course it's a low-risk merge so there isn't anything _wrong_ with
merging to 4.4, but, 4.4 still has the old style logic for choosing $JAVA_HOME,
so those changes would have to be cherry-picked too, and that implies changes
to preferential status of 1.7 vs 1.6 for 4.4.........so I would guess, just
leave it for 4.5.
> centos cloudstack-usage script does not always pass along $JAVA_HOME
> --------------------------------------------------------------------
>
> Key: CLOUDSTACK-7476
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-7476
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Components: Usage
> Affects Versions: 4.5.0
> Environment: secured centos/redhat
> Reporter: Leo Simons
> Fix For: 4.5.0
>
>
> /etc/init.d/cloudstack-usage finds a $JAVA_HOME and makes sure the
> environment variable is set, then assumes this variable will be picked up by
> JSVC.
> However, on a secured environment (selinux w/ env_reset enabled in sudoers),
> the runuser command that is invoked by the daemon() function does not pass
> along environment variables, so $JAVA_HOME is empty, and JSVC falls back to
> its default behavior, which may not find java or may not find the intended
> java.
> The simple solution is to pass -home to JSVC, passing it on the command line
> instead of as an environment variable.
> I'll provide a patch.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)