Skip to content

Conversation

@vadmeste
Copy link
Member

@vadmeste vadmeste commented Mar 5, 2020

A new command is added to mc admin user command called svc
and that's to create new services accounts from existing users.

It is also possible to show credentials of the existing service
accounts.

mc admin user svc generate and mc admin user svc show

@vadmeste vadmeste changed the title [WIP] Add sv command to support service accounts [WIP] Add svc command to support service accounts Mar 10, 2020
@vadmeste vadmeste force-pushed the service-account branch 2 times, most recently from 7ef6ed0 to ace55aa Compare March 21, 2020 22:17
@vadmeste vadmeste changed the title [WIP] Add svc command to support service accounts Add svc command to support service accounts Mar 21, 2020
@vadmeste
Copy link
Member Author

Not WIP anymore

@vadmeste vadmeste changed the title Add svc command to support service accounts [WIP] Add svc command to support service accounts Mar 21, 2020
@vadmeste vadmeste changed the title [WIP] Add svc command to support service accounts [WIP] Add sa command to support service accounts Mar 22, 2020
A new command is added to `mc admin user` command called svc
and that's to create new services accounts from existing users.

It is also possible to show credentials of the existing service
accounts.

`mc admin user sa generate PARENT-USER [POLICY_FILE]`
`mc admin user sa show SERVICE_ACCOUNT_ACCESS_KEY`
@vadmeste vadmeste changed the title [WIP] Add sa command to support service accounts Add sa command to support service accounts Mar 23, 2020
@vadmeste
Copy link
Member Author

Not WIP. SA command supports empty policy now

vadmeste and others added 2 commits March 24, 2020 11:44
Copy link
Member

@harshavardhana harshavardhana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, lets get this reviewed by @abperiasamy everything else is working fine.

@harshavardhana
Copy link
Member

@vadmeste can you get this reviewed from @abperiasamy ?

@dvaldivia dvaldivia mentioned this pull request Apr 9, 2020
6 tasks
@harshavardhana
Copy link
Member

Closing this PR as this is deprecated and not going to be implemented, as this functionality completely moves to mcs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants