[GitHub] [incubator-kyuubi] zhouyifan279 edited a comment on issue #1204: [FEATURE] Enhance zookeeper authentication and acls of Kyuubi HA module

[GitBox <gi...@apache.org>]

zhouyifan279 edited a comment on issue #1204:
URL: https://github.com/apache/incubator-kyuubi/issues/1204#issuecomment-942116481


   > It looks fine to me. However, can we merge some of these configurations, it is now very hard to explain and use.
   > 
   > ```
   > kyuubi.ha.zookeeper.acl.enabled=true
   > kyuubi.ha.zookeeper.auth.sasl.kerberos=false
   > kyuubi.ha.zookeeper.auth=digest:hue:***
   > kyuubi.ha.zookeeper.acl.engine.enabled=true
   > ```
   > 
   > cc @zhouyifan279, do you have any idea, if we can add some unit tests to test with a kerberied zookeeper and acls
   
   As we already have `org.apache.kyuubi.KerberizedTestHelper` to setup a KDC, and zookeeper integrates kerberos using JAAS, it should be easy to setup an kerberied embedde zookeeper server.
   
   @wForget , would you mind to add these test cases ? 
   
   I'm also glad to do the work if you have no time.


-- 
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: dev-unsubscribe@kyuubi.apache.org

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