brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aledsage <...@git.apache.org>
Subject [GitHub] incubator-brooklyn pull request: brooklyn.properties permissions c...
Date Fri, 08 Aug 2014 17:38:49 GMT
GitHub user aledsage opened a pull request:

    https://github.com/apache/incubator-brooklyn/pull/115

    brooklyn.properties permissions check

    This change will cause brooklyn to fail to start if brooklyn.properties do not have permission
x00 (e.g. 400, 600 or 700).
    
    e.g.:
    
        brooklyn launch --localBrooklynProperties localBrooklyn.properties 
         _                     _    _             
        | |__  _ __ ___   ___ | | _| |_   _ _ __ (R)
        | '_ \| '__/ _ \ / _ \| |/ / | | | | '_ \ 
        | |_) | | | (_) | (_) |   <| | |_| | | | |
        |_.__/|_|  \___/ \___/|_|\_\_|\__, |_| |_|
                                      |___/             0.7.0-SNAPSHOT
        
        2014-08-08 18:31:48,829 ERROR Startup error: Invalid permissions for file localBrooklyn.properties;
expected 600 but was -rw-r--r--
        Startup error: Invalid permissions for file localBrooklyn.properties; expected 600
but was -rw-r--r--
    
    Note the "Adds FileUtil" commit is a duplicate of code in PR #114. Suggest we merge that
one first (depending which one folk get around to reviewing first).

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

    $ git pull https://github.com/aledsage/incubator-brooklyn fix/brooklynProperties-permissions-check

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

    https://github.com/apache/incubator-brooklyn/pull/115.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 #115
    
----
commit e04d5d019f8675c83c7cbe7f4c04d6daf5de57f8
Author: Aled Sage <aled.sage@gmail.com>
Date:   2014-08-08T16:59:47Z

    Adds FileUtil (see PR #114)

commit 0d235c050ab45b5bdef153f9fec1bec60fde7ae9
Author: Aled Sage <aled.sage@gmail.com>
Date:   2014-08-08T17:25:46Z

    BROOKLYN-15: enforce brooklyn.properties permissions
    
    - brooklyn.properties must be something like 600; otherwise
      brooklyn will fail to start

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message