[ 
https://issues.apache.org/jira/browse/HBASE-13767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14559612#comment-14559612
 ] 

Enis Soztutar commented on HBASE-13767:
---------------------------------------

+1. 

Can you please do these changes as well: 
{code}
@@ -41,70 +39,69 @@ import org.apache.zookeeper.WatchedEvent;
  * You may also specify -D to set options
  *    "hbase.zookeeper.quorum"    (it should be in hbase-site.xml)
  *    "zookeeper.znode.parent"    (it should be in hbase-site.xml)
+ *
+ * Use -set-acls to set the ACLs, no option to erase ACLs
  */
 @InterfaceAudience.Private
 public class ZkAclReset extends Configured implements Tool {
   private static final Log LOG = LogFactory.getLog(ZkAclReset.class);
{code}
and
{code}
   public static void main(String[] args) throws Exception {
-    System.exit(ToolRunner.run(new Configuration(), new ZkAclReset(), args));
+    System.exit(ToolRunner.run(HBaseConfiguration.create(), new ZkAclReset(), 
args));
   }
{code}

The latter is needed so that the jaas conf is automatically applied. 

> Allow ZKAclReset to set and not just clear ZK ACLs
> --------------------------------------------------
>
>                 Key: HBASE-13767
>                 URL: https://issues.apache.org/jira/browse/HBASE-13767
>             Project: HBase
>          Issue Type: Bug
>          Components: Operability, Zookeeper
>    Affects Versions: 2.0.0, 1.2.0
>            Reporter: Matteo Bertozzi
>            Assignee: Matteo Bertozzi
>            Priority: Trivial
>             Fix For: 2.0.0, 0.98.13, 1.0.2, 1.2.0, 1.1.1
>
>         Attachments: HBASE-13767-v0.patch, HBASE-13767-v1.patch, 
> HBASE-13767-v2.patch
>
>
> The ZKAclReset tool allows to clear ZK ACLs, which is useful if you are 
> migrating from a secure to unsecure cluster setup.
> If you want to make sure that your znode ACLs are correct, a -set-acls 
> option, which allows to enforce the proper ACLs on the znodes in a secure 
> setup, can be useful too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to