Skip to content
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 MinIO admin API support. #1076

Merged
merged 2 commits into from
Feb 12, 2021

Conversation

balamurugana
Copy link
Member

No description provided.

minio/minioadmin.py Outdated Show resolved Hide resolved
@balamurugana balamurugana marked this pull request as draft February 2, 2021 19:16
@balamurugana balamurugana force-pushed the Add-MinIO-admin-API-support branch 5 times, most recently from f47cb44 to d665965 Compare February 3, 2021 05:13
@balamurugana balamurugana marked this pull request as ready for review February 3, 2021 05:14
minio/minioadmin.py Outdated Show resolved Hide resolved
@krisis
Copy link
Member

krisis commented Feb 4, 2021

@balamurugana it would be useful to have a README introducing this package and a couple of simple examples to demonstrate usage like we have for the minio package .

@kannappanr kannappanr merged commit 1ac8c82 into minio:master Feb 12, 2021
@balamurugana balamurugana deleted the Add-MinIO-admin-API-support branch February 17, 2021 15:03
@torbendury
Copy link

Is there any documentation available on how to use this MinioAdmin object?

@balamurugana
Copy link
Member Author

@torbendury If you know how to use mc admin command line, you can directly refer https://github.com/minio/minio-py/blob/master/minio/minioadmin.py

@torbendury
Copy link

@balamurugana That's what I did, but searching through function definitions doesn't feel right. At least a refer on the Python SDK docs to this file would be great, so that other who wonder about the admin API don't wonder how to access it.

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.

None yet

5 participants