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

MinioAdmin class uses deprecated mc commands #1294

Closed
Nox-404 opened this issue Aug 14, 2023 · 2 comments
Closed

MinioAdmin class uses deprecated mc commands #1294

Nox-404 opened this issue Aug 14, 2023 · 2 comments

Comments

@Nox-404
Copy link

Nox-404 commented Aug 14, 2023

Hi,
I've been looking at the MinioAdmin class that wraps the mc admin command and noticed some deprecation...

  • mc admin policy add -> mc admin policy create
  • mc admin policy set -> mc admin policy attach
  • mc admin policy unset -> mc admin policy detach

I did not check if #1291 also address these problems too, but since it's not available yet the problem remain

@ctengel
Copy link

ctengel commented Aug 14, 2023

Same issue as #1293 ?

I tested #1277 by running pip3 install https://github.com/minio/minio-py/archive/refs/heads/master.zip (getting latest from git vs using 7.1.15) and it seemed to resolve the issue, at least for my use case.

@Nox-404
Copy link
Author

Nox-404 commented Aug 14, 2023

oh it seems I did not check thoroughly !
I got a patch on my end for that, so I'll just have to wait for the release in my distro to remove it.

@Nox-404 Nox-404 changed the title MinioAdmin class uses deprecated mc commands MinioAdmin class error handling is too opaque Aug 14, 2023
@Nox-404 Nox-404 changed the title MinioAdmin class error handling is too opaque MinioAdmin class uses deprecated mc commands Aug 14, 2023
@Nox-404 Nox-404 closed this as completed Aug 14, 2023
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

No branches or pull requests

2 participants