Github user drigodwin commented on a diff in the pull request:

    https://github.com/apache/brooklyn-docs/pull/174#discussion_r113575475
  
    --- Diff: guide/ops/security-guidelines.md ---
    @@ -72,7 +72,10 @@ root login.
     ### VM Users
     
     It is strongly discouraged to use the root user on VMs being created or 
managed by Brooklyn.
    +SSH access should be delegated on rare cases such as initial Apache 
Brooklyn setup and other maintenance occasions.
     
    +Avoid putting config a lot of options in `etc/brooklyn.cfg` which one 
would later need to ssh and change.
    +Keep location configuration it is preferable to use [Locations in the 
Catalog](./catalog/#locations-in-the-catalog).
    --- End diff --
    
    I think these two lines need rewording but I'm not sure what they're 
adding. `etc/brooklyn.cfg` is a config file in the Karaf Brooklyn distro which 
is not currently the default. I'm not sure we're advising people put locations 
in there anywhere.


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