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

[Bug]: [privilege_group] The built in privilege groups are not listed using interface "list_privilege_groups()" #37887

Closed
1 task done
binbinlv opened this issue Nov 21, 2024 · 5 comments
Assignees
Labels
kind/bug Issues or changes related a bug triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@binbinlv
Copy link
Contributor

binbinlv commented Nov 21, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Environment

- Milvus version: master-20241120-c73ca14a-amd64
- Deployment mode(standalone or cluster):both
- MQ type(rocksmq, pulsar or kafka):    all
- SDK version(e.g. pymilvus v2.0.0rc2): 2.5.0rc126
- OS(Ubuntu or CentOS): 
- CPU/Memory: 
- GPU: 
- Others:

Current Behavior

The built in privilege groups are not listed using interface "list_privilege_groups()"

>>> role.list_privilege_groups()
[]

Expected Behavior

list all the built in privilege groups

Steps To Reproduce

from pymilvus import Role
from pymilvus import connections
connections.connect()
role = Role("newRole")
role.create()
role.list_privilege_groups()
[]

Milvus Log

No response

Anything else?

No response

@binbinlv binbinlv added kind/bug Issues or changes related a bug triage/accepted Indicates an issue or PR is ready to be actively worked on. labels Nov 21, 2024
@binbinlv
Copy link
Contributor Author

/assign @shaoting-huang
/unassign @yanliang567

@sre-ci-robot
Copy link
Contributor

@binbinlv: GitHub didn't allow me to assign the following users: shaoting-huang.

Note that only milvus-io members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @shaoting-huang
/unassign @yanliang567

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@shaoting-huang
Copy link
Contributor

/assign

@binbinlv
Copy link
Contributor Author

The built in roles could be listed using "utility.list_roles(True)", so I think here should keep the consistent. And it will be more convenient for users to get the detail of the built in privilege groups.

>>> utility.list_roles(True)
RoleInfo groups:
- RoleItem: <role_name:admin>, <users:()>
- RoleItem: <role_name:public>, <users:()>

sre-ci-robot pushed a commit that referenced this issue Nov 21, 2024
issue: #37031
also fix issues: #37843,
#37842,
#37887

Signed-off-by: shaoting-huang <shaoting.huang@zilliz.com>
@binbinlv
Copy link
Contributor Author

Verified and fixed:
pymilvus: 2.5.0rc126
milvus: master-20241122-bc131a91-amd64

>>> res = role.list_privilege_groups()
>>> len(res)
9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues or changes related a bug triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

4 participants