-
Notifications
You must be signed in to change notification settings - Fork 550
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
Add mc admin accesskey
commands
#5038
Conversation
@taran-p can you resolve the conflicts ? and when you're done.. please tag someone again to take a look |
f8a3562
to
d7f3188
Compare
Testing with latest MinIO master and seeing below
Just created user not getting listed and |
@shtripat |
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.
Works fine. One suggestion on below
./mc admin accesskey edit m1 user1
Successfully edited access key `user1`.
while edit, it should ask for editing at least one value, otherwise its just making a server round trip without any changes.
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.
lgtm
Community Contribution License
All community contributions in this pull request are licensed to the project maintainers
under the terms of the Apache 2 license.
By creating this pull request I represent that I have the right to license the
contributions to the project maintainers under the Apache 2 license.
Description
Adds
mc admin accesskey
commands that are equivalent tomc idp ldap accesskey
commands, but for builtin users.mc idp ldap accesskey create-with-login
is not brought over as it is unnecessary for builtin users.Motivation and Context
mc idp ldap accesskey
commands currently have no builtin equivalents.How to test this PR?
Use commands on builtin users. For
mc admin accesskey list
, minio/minio#20381 is necessary.Types of changes
Checklist:
commit-id
orPR #
here)