petedannemann
commented
10 months ago - bump kafka-go to include acl apis
- add acl interfaces and aclinfo type stub
- pull latest kafka-go and use kafka-go aclresource type
- wip
- fix test
- fix typos
- get acls working
- getacls working
- upgrade cobra to latest
- finish separating get into separate subcommands
- remove unneeded variables
- wip
- pr feedback
- Revert "upgrade cobra to latest"
- use getCliRunnerAndCtx in get acls
- more consistent variable names
- custom cobra type
- bring in new kafka-go
- support resource pattern type
- add support for acloperationtype and remove options for unknown
- improve descriptions
- support permissiontype and host filters
- add resource name filter and fix permission type formatting
- support principal filtering
- improve docs
- add examples
- remove comment
- remove TODOs that are complete
- remove TODOs that are complete
- update README
- fix test
- wip
- fix error handling
- error handling for zk
- more consistent error msg
- clean up createacl
- add TestBrokerClientCreateACLReadOnly
- improve zk tests
- run acl tests in ci
- enable acls for kafka 2.4.1 in ci
- fix zk tests
- skip TestBrokerClientCreateACLReadOnly on old versions of kafka
- try to debug
- handle nested errors from createacls
- operations -> operation
- operations -> operation
- remove setting log level in test
- clean up allowed types in help command
- fix merge conflict
- fix test
- add json annotations
- bump kafka-go to version on main
- wip
- basic tests
- start on getusers cmd
- add json annotations
- get users working
- wip
- add todos and fix type annotaitons
- improve test
- use CanTestBrokerAdminSecurity to feature flag test
- update README