-
Notifications
You must be signed in to change notification settings - Fork 552
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
rpk: move acl / acl user to /security #17664
Conversation
This movement is a soft deprecation to the previous command. The old command will work as intended but print a WARN message (only visible with -v) showing that the command is deprecated.
This is done in _some_ commands, not all. We want to make sure that both (old and new) still work.
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/47417#018eab71-0ed4-4e35-9b7f-bacc4b19d269 ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/47423#018eabfc-8efa-4d5d-90e4-8a742eb89071 |
When you create a role, you must bind or associate ACLs to it before it can be | ||
used. You can create / delete / list ACLs for that role with "<name>" in the | ||
--allow-role and --deny-role flags. Note that the wildcard role name '*' is not | ||
permitted here. For example 'rpk acl create --allow-role '*' ...' will produce |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should say 'rpk security acl create --allow-role '*' ...'
?
|
||
Check --help-operations to see which operations are required for which | ||
requests. In flag form to set up a general producing/consuming client, you can | ||
invoke 'rpk acl create' three times with the following (including your |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
similarly here 'rpk security acl create'
?
Because we want to tag rc-1, I'll go ahead with the merge and the minor nits above could come in a follow-up PR. |
This movement is a soft deprecation to the previous command. The old command will work as intended
but print a WARN message (only visible with -v)
showing that the command is deprecated.
Backports Required
Release Notes
Improvements
rpk acl
has been moved torpk security acl
. Old command is soft deprecated.rpk acl user
has been moved torpk security security
. Old command is soft deprecated.