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

Unable to add ACLs using kfk acls --add but with kfk users --alter --add-acl #87

Closed
arpan57 opened this issue Jul 28, 2021 · 3 comments
Closed
Assignees
Labels
duplicate This issue or pull request already exists
Milestone

Comments

@arpan57
Copy link

arpan57 commented Jul 28, 2021

Hi,

I am trying to add an ACL like following

kfk acls --add --allow-principal User:secure-writer --operation Write --topic secure-topic -c my-cluster -n secure-ns
Hitting return key I get a message
kafkauser.kafka.strimzi.io/secure-writer configured.

But when I describe the user using kfk users --describe --user secure-writer -n secure-ns -c my-cluster I don't see anything for Authorization added in the Spec portion of the User.

Similarly if I list the acls using
kfk acls --list -n secure-ns -c my-cluster I don't see any entry for the secure-topic or for secure-writer.

Now I am trying to add ACLs using the kfk users

kfk users --user secure-writer --alter --authorization-type simple --add-acl --operation Write --resource-type topic --resource-name secure-topic -c my-cluster -n secure-ns

This works.
I can see the ACLs got updated for the user. I can see it updated under kfk user --describe as well as in kfk acls --list also.

Am I missing any parameters while creating the ACLs using kfk acls? or it's a bug?

Versions
Strimzi CLI 0.1.0a57
Strimzi 0.20.1
Kubectl v1.16.15

Regards,
Arpan

@arpan57 arpan57 changed the title Unable to add ACLs using kfk acls Unable to add ACLs using kfk acls --add but works with kfk users --alter --add-acl Jul 28, 2021
@arpan57 arpan57 changed the title Unable to add ACLs using kfk acls --add but works with kfk users --alter --add-acl Unable to add ACLs using kfk acls --add but with kfk users --alter --add-acl Jul 28, 2021
@mabulgu mabulgu self-assigned this Aug 11, 2021
@mabulgu mabulgu added the bug Something isn't working label Aug 11, 2021
@mabulgu mabulgu added this to the Beta Launch milestone Aug 11, 2021
@mabulgu
Copy link
Member

mabulgu commented Sep 25, 2021

I think this is because we dont set authorization type while running kfk acls --add --allow-principal User:secure-writer --operation Write --topic secure-topic -c my-cluster -n secure-ns.

That should do it automatically. The following is an already open issue for this: #33

Will be fixed in the next release!

@mabulgu mabulgu added duplicate This issue or pull request already exists and removed bug Something isn't working labels Sep 25, 2021
@mabulgu
Copy link
Member

mabulgu commented Sep 25, 2021

Closing this one as well since I will fix this in #33.

@mabulgu mabulgu closed this as completed Sep 25, 2021
@mabulgu
Copy link
Member

mabulgu commented Sep 26, 2021

Fix is released with the new version: https://github.com/systemcraftsman/strimzi-kafka-cli/releases/tag/0.1.0-alpha58

Run the following to upgrade your Strimzi CLI version:

sudo pip install strimzi-kafka-cli --upgrade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
Development

No branches or pull requests

2 participants