Closed Cottand closed 1 year ago
cc @kvaster possibly related to recent PRs? Or the doc needs changes?
I'm investigating. It's look really strange.
Yes. It's really related to my changes, I will make one more PR in a 30 minutes. The problem is that I've introduced incompatibility with previous setups. From now you should run either --controller
or --node
or both of them the same time.
if this is the result of a breaking change, I would ideally expect
thanks!
It was not supposed to be a breaking change. I've made a PR which fixes the problem. It was supposed that previous installs would work without any changes.
I see, no worries then. In that case I would appreciate some docs on what --controller
or --node
do and other available options
It was a big refactoring for running driver in kubernetes. Controller server should be running separate of node server. Node server is a daemon which runs on all nodes which can mount seaweedfs and controller should be just fail safe and HA.
It's all about CSI.
to achieve the same behaviour as before - can I use both options on all boxes safely? Or will the controllers need to speak to each other/will that increase gossip somehow?
ie, is the example Nomad deployment unchanged (I might need to run a controller separately) or do I have better options now, for HA or performance?
edit - I still get nil dereference when using both options on my existing setup
@chrislusf you marked as complemeted but in https://github.com/seaweedfs/seaweedfs-csi-driver/pull/134 you did not update the Nomad example (but updated the helm charts) - do the default options for Nomad remain unchanged?
Yes. Default options remain unchanged now - as it should be.
I am running this as a CSI plugin to Nomad. I followed this example, except
The CSI plugin fails on any Nomad client (any pod) so I think the trace is not specific to the host machine, althoguh all my machines are configured very similarly. Version is
latest
for the CSI image, 3.55 for filer, volumes, master etc.Logs:
CSI plugin job:
Let me know if I should provide more info.