Fix NATS configuration guide as per CRD#1073
Fix NATS configuration guide as per CRD#1073kailash360 wants to merge 1 commit intomilvus-io:masterfrom
Conversation
|
thx for your debugging of docs, |
|
Thanks for the update @chyezh. But I am currently using Milvus version 2.6.x and it seems that it is able to take up NATS config using just So, has the removal of NATS not been completed yet? If not, by when do we plan to completely let go of NATS from Milvus? Also, I will appreciate if you can throw some light on why we are removing NATS. |
It has been removed indeed, maybe the configuration didn't take effect on your cluster. |
Fixes #1072
This pull request fixes the NATS configuration guide for Milvus Operator.
Existing Behaviour
There is mistmatch between the CRD schema for NATS and the manifest given in the document guide. This causes error while starting milvus with the current guide.
New Behaviour
The manifest documentation for NATS configuration is now matching the CRD. The existing fields are now placed under
spec.config.natsmqwhich allows to inject additional fields.