-
Notifications
You must be signed in to change notification settings - Fork 801
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated rootpath validation method in zookeeper coordinator #422
Conversation
Validate rootpath with Exists instead of Create to avoid 'zk: not authenticated' error
Hi @toddpalino, I tried to solve the issue by digging into the ACL part in samuel/go-zookeeper repo, and found that it does not currently support SASL authentication, where at least a username should be returned in AuthACL (see "https://ihong5.wordpress.com/tag/zookeeper-acl/"). I tried to modify Burrow and pass on SASL credentials using DigestACL but it didn't work. If anyone is interested in fixing the problem I am happy to help & learn. Thanks! |
My apologies for all the messy small commits. I haven't figured out how to test build and coverage locally as I cannot get past the 'use of internal package is not allowed' restriction in Golang. |
@cluyihunter If you move your src over to |
@maxtuzz Yes I have github.com/linkedin/Burrow in src under my gopath, but still the error persists whenever I try testing (use of internal package is not allowed) |
Validate rootpath with Exists instead of Create to avoid 'zk: not authenticated' error.