[GitHub] [accumulo-website] EdColeman commented on a diff in pull request #371: Add zoo-info-viewer --print-acls usage, document expected ZooKeeper ACLs

["EdColeman (via GitHub)" <gi...@apache.org>]

EdColeman commented on code in PR #371:
URL: https://github.com/apache/accumulo-website/pull/371#discussion_r1094829015


##########
_docs-2/troubleshooting/zookeeper.md:
##########
@@ -3,6 +3,34 @@ title: ZooKeeper
 category: troubleshooting
 order: 7
 ---
+## ZooKeeper ACLs
+
+Accumulo requires full access to nodes in ZooKeeper under the /accumulo path.  The ACLs can be examined using the
+ZooKeeper cli `getAcl` and modified with `setAcl` commands.  With 2.1.1, the zoo-info-viewer utility has an option
+that will print all of the ACLs for the nodes under `/accumulo/[INSTANCE_ID]` (See the [zoo-info-viewer]).  
+To run the utility, only ZooKeeper needs to be running. If hdfs is running, the instance id can be read from hdfs, 
+or it can be entered with the zoo-info-viewer command --instanceId option.  Accumulo management processes 
+*do not* need to be running. This allows checking the ACLs before starting an upgrade. 
+

Review Comment:
   Added in 12938808



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org