StephenSorriaux / ansible-kafka-admin

Manage your topic's configuration (partitions, replication factor, parameters), ACLs, quotas, users and get stats, without any effort with this library. It does not use the Kafka scripts and does not require ssh connection to the remote broker.
Apache License 2.0
150 stars 46 forks source link

fix request to get full list of acls #105

Closed mrandreyeff closed 3 years ago

mrandreyeff commented 3 years ago

Checked at kafka 2.6 only. Issue: the describe_acls returns acl_resource_found that is limited only to subset of host: '*' literals, i.e. if there is an IP, this record is ignored. This caused adding already existing ACLs (nothing too bad, but just wrong status and some extra calls) and, in case mark_others_as_absent: True, this causes to malfunction and skip of all records, which have an IP address.

ryarnyah commented 3 years ago

LGTM