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

Implement __eq__ and __hash__ for ACL objects #1955

Open
wants to merge 2 commits into
base: master
from

Conversation

@TylerLubeck
Copy link

TylerLubeck commented Nov 15, 2019

This allows for direct comparison of ACL objects, and allows us to create sets of them

I've found it useful for some work I'm doing to allow us to manage ACLs in some source-controlled way

  1. read the 'expected' ACLs from the source controlled file, create ACL objects
  2. fetch all 'current' ACLs from kafka
  3. create set objects of both
  4. Do set operations to determine what commands I need to run to bring kafka in sync with the expected ACL configurations

This change is Reviewable

TylerLubeck added 2 commits Nov 15, 2019
This allows for direct comparison of ACL objects, and allows us to
create `set`s of them
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.