GitHub user aledsage opened a pull request:

    https://github.com/apache/brooklyn-dist/pull/24

    Fix RPM log dir

    @alrick87 @rdowner can you take a look please? Really want this fix in 
before 0.9.0 release candidate!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/aledsage/brooklyn-dist fix/rpm-log-dir

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-dist/pull/24.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #24
    
----
commit 691a0bdc3332a147d5d4b1e4e1ec7b2a995bb12e
Author: Aled Sage <[email protected]>
Date:   2016-03-18T23:12:41Z

    RPM: /var/log/amp permission is 700
    
    The log file could potentially contain credentials (depending on how
    Brooklyn is used) so make sure it’s not visible/readable.

commit d55762d554e229fdfc0adc17b744a437ed3dea00
Author: Aled Sage <[email protected]>
Date:   2016-03-18T23:14:36Z

    Fix brooklyn.service: include JAVA_OPTS
    
    Note the careful use of $JAVA_OPTS rather than ${JAVA_OPTS} so that 
    it is passed as multiple command line options, rather than one.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to