minio / mc

Unix like utilities for object store
https://min.io/download
GNU Affero General Public License v3.0
2.88k stars 552 forks source link

iam export:designate zip file name #4995

Closed dormanze closed 2 months ago

dormanze commented 4 months ago

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

Motivation and Context

How to test this PR?

image

Types of changes

Checklist:

klauspost commented 4 months ago

It seems a bit contrary to the usual design that the parameter is always assumed to be local path and not an alias.

Adding a -o=/tmp/myminio-iam.zip that allows to override the output path seems more appropriate to me.

dormanze commented 4 months ago

It seems a bit contrary to the usual design that the parameter is always assumed to be local path and not an alias.

Adding a -o=/tmp/myminio-iam.zip that allows to override the output path seems more appropriate to me.

This is a good idea. I added a new flag. Please review again.

dormanze commented 3 months ago

@harshavardhana Please review my pr again.

klauspost commented 2 months ago

@dormanze I presume it is a script, but could you please turn off the automatic merge with master? It is very noisy and we need to manually re-run tests every time.

dormanze commented 2 months ago

@dormanze I presume it is a script, but could you please turn off the automatic merge with master? It is very noisy and we need to manually re-run tests every time.

My bad, This is not a script, I will look at my unclosed pr while browsing the community, manually trigger an update, sorry to bother you, I will stop this operation.