minio / minio-py

MinIO Client SDK for Python
https://docs.min.io/docs/python-client-quickstart-guide.html
Apache License 2.0
851 stars 325 forks source link

Refactor MinioAdminClient using HTTP client #1291

Closed pbrw closed 1 year ago

pbrw commented 1 year ago

Attempt to solve these issues #829 #1243

Now it should be straightforward to implement the rest of MinIO Admin API

pbrw commented 1 year ago

I rewrote it according to your requests. At the moment I commented all the MinioAdmin commands with mc and I will rewrite them with HTTP in subsequent commits

balamurugana commented 1 year ago

I rewrote it according to your requests. At the moment I commented all the MinioAdmin commands with mc and I will rewrite them with HTTP in subsequent commits

@pbrw I sent a PR https://github.com/pbrw/minio-py/pull/1 to your fork instead of lengthy review. Please review and merge it in your fork.

balamurugana commented 1 year ago

@pbrw I sent you PR https://github.com/pbrw/minio-py/pull/3 and reverted prometheus_create() addition. Few APIs are removed and site replication APIs are added. I also increased minor version to indicate backward compatibility.

balamurugana commented 1 year ago

@pbrw Please fix CI failure

jessebot commented 11 months ago

how do you specify credentials now? Previously MinioAdmin took an alias.