-
Notifications
You must be signed in to change notification settings - Fork 49
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
[milvus-4.1.18] Enhancement: Configurable User limit settings for Milvus #63
[milvus-4.1.18] Enhancement: Configurable User limit settings for Milvus #63
Conversation
/assign @zwd1208 |
@Rachit-Chaudhary11 Please also sign-off this commit. And you may also need to rebase after the last PR is merged, and make sure the Chart Version is bumped. |
Signed-off-by: Rachit Chaudhary - r0c0axe <Rachit.Chaudhary@walmart.com>
463b64b
to
7c8702c
Compare
# Conflicts: # charts/milvus/Chart.yaml
/lgtm |
@Rachit-Chaudhary11 We bump the chart version in every PR. |
Signed-off-by: Rachit Chaudhary - r0c0axe <Rachit.Chaudhary@walmart.com>
@haorenfsa please check, I have bumped up the milvus chart version |
/assign @zwd1208 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: Rachit-Chaudhary11 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Author - @Rachit-Chaudhary11
What this PR does / why we need it:
This update will allow for better scalability and flexibility in managing the users access to milvus instance. Presently this limit is set to 100 users per milvus instance and there is no way to increase / decrease the users limit to milvus instance.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[mychartname]
)