Skip to content
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

Add option to recursively set ACLs #740

Open
marcoppenheimer opened this issue Feb 3, 2024 · 0 comments
Open

Add option to recursively set ACLs #740

marcoppenheimer opened this issue Feb 3, 2024 · 0 comments

Comments

@marcoppenheimer
Copy link

What would you like to be added

As of ZOOKEEPER-1908 the CLI supports running setAcl recursively

I would like the Kazoo client to be able to do the same, to avoid having to issue multiple calls for every child zNode.

Something like: Kazoo.client.set_acls(path=path, acls=acls, recursive=True)

Why is this needed

It's not uncommon to have a unique SASL (or other scheme) credential + ACLs assigned to their own parent zNode. For example, the chroot for Kafka.

Right now, if I'm not mistaken, if we want to change ACLs for that parent zNode, we have to:

  1. Recursively get all child zNodes from the parent
  2. Call Kazoo.client.set_acl for each one

This is cumbersome, and would be great if we could do it recursively from the get-go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants