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

Support dynamic change user.yaml ? #129

Closed
anhnch30820 opened this issue Jun 4, 2024 · 9 comments
Closed

Support dynamic change user.yaml ? #129

anhnch30820 opened this issue Jun 4, 2024 · 9 comments

Comments

@anhnch30820
Copy link

I deployed Milvus with config here

apiVersion: milvus.io/v1beta1
kind: Milvus
metadata:
  name: my-release
  labels:
    app: milvus
spec:
  mode: cluster
  dependencies: {}
  components: 
    updateConfigMapOnly: true
  config:
    common:
      ttMsgEnabled: false

then I changed ttMsgEnabled: true and kubectl apply. But I can't search, insert or delete with error Deadline Exceeded.
So How can it work?

@haorenfsa
Copy link
Collaborator

Hi @anhnch30820 Which version of Milvus are you using? This only works for Milvus v2.4.0 or later version. And it takes some time (about a minute or so) for kubelet to change the file and for Milvus to reload config.

@anhnch30820
Copy link
Author

anhnch30820 commented Jun 5, 2024

@haorenfsa
I use version 2.4.4 with operator 0.9.16. My workflow:

Create milvus with ttMsgEnabled: false --> change ttMsgEnabled: true and apply --> create collection --> create index --> load collection.

But, load collection stuck at 0% and I waited for more than 30 minutes. I enter pod and see config

ttMsgEnabled: true in file user.yaml
ttMsgEnabled: false in file milvus.yaml

I don't know how to fix?

@haorenfsa
Copy link
Collaborator

@anhnch30820 Milvus will prioritize user.yaml if it's different from milvus.yaml.

Could you please provide the log of the Milvus Pod?

@anhnch30820
Copy link
Author

anhnch30820 commented Jun 7, 2024

@haorenfsa Here is the log milvus-proxy
milvus-milvus-proxy-6c98fb49c4-622fp_proxy.log

@haorenfsa
Copy link
Collaborator

Hi @anhnch30820, it's an known issue in milvus milvus-io/milvus#33461. Team Milvus plan to fix this in v2.4.5

@anhnch30820
Copy link
Author

@haorenfsa I hope it will be released soon, thank you.

@haorenfsa
Copy link
Collaborator

@anhnch30820 milvus v2.4.5 is released, please help us verify if you got time.

@anhnch30820
Copy link
Author

@haorenfsa it doesn't work.

@haorenfsa
Copy link
Collaborator

track it in milvus-io/milvus#33461

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