-
Notifications
You must be signed in to change notification settings - Fork 553
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: fix mixed use of backcompat flags with -X #16278
Conversation
/dt |
new failures in https://buildkite.com/redpanda/redpanda/builds/44247#018d3db6-daf7-47b7-a362-1edf1f25f0e9:
new failures in https://buildkite.com/redpanda/redpanda/builds/44247#018d3dc6-b87e-478d-9299-106f8bf35882:
|
We were giving priority to old back compatibility flags (like --user, --password) over the new configuration (-X) flags. So if a user used both, the old ones got priority. This introduced a bug in rpk acl user create where we have a flag that collides with an old flag name: --password.
ccadd4e
to
16835f6
Compare
/dt |
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/44259#018d3e8b-3c88-4e47-9d29-67508bff2d66 ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/44259#018d4122-91b5-44c9-aaa9-89642a0bdc9d |
Unrelated error: https://buildkite.com/redpanda/redpanda/builds/44291#018d40e1-4549-41ce-b2d4-d91f0caed26a/6570-6895 Redpanda build timed out during RP unit test execution |
/backport v23.3.x |
/backport v23.2.x |
Note: backport to v23.1.x is not needed since the issue doesn't exist in that version. |
We were giving priority to old back compatibility
flags (like --user, --password) over the new
configuration (-X) flags. So if a user used both,
the old ones got priority.
This introduced a bug in rpk acl user create
where we have a flag that collides with an old
flag name: --password.
Fixes #16251
Backports Required
Release Notes
Bug Fixes
--password
flag could not be used along with the new configuration flag-X pass
in clusters where basic authentication was enabled.