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

Driver refactoring and cleanup, part 2 #132

Merged
merged 11 commits into from
Aug 7, 2023
Merged

Conversation

kvaster
Copy link
Contributor

@kvaster kvaster commented Aug 7, 2023

Refactoring, part 2

  • Use SIGTERM for shutting down weed mount
    It seems that running umount on mount point does not close weed mount process or trigger any event for this. At least it works this way on my linux box with latest code. i.e. umount is issued on mount point, mount point becomes free, but weed process continues to work and only SIGTERM tells this process to end.
  • Adjust timeouts
  • Use stronger mount point check
  • Stop gracefully
  • Cleanup active mounts
  • Yaml files consistent naming
  • Allow multiple controllers (replicas) and separate controller server from node server
  • Allow to disable attacher, cause it's really usefull for this driver
  • Liveness probes

https://kubernetes-csi.github.io/docs/volume-expansion.html

CSI Resizer should be always run even if not implemented. Also it should run only against controller.
Volume resize on node rpc call is handled by kubelet.
For now the only job for controller's volume resize is to make it two step and propagate changes to node.
* Separate NodeServer and ControllerServer
* Allow leader election for sidecars and replica count fot controller component
* Liveness probes
* Allow disabling attacher, cause this functionality is not used in reality
@chrislusf chrislusf merged commit f294627 into seaweedfs:master Aug 7, 2023
2 of 5 checks passed
@kvaster kvaster deleted the refactor branch August 7, 2023 18:45
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

Successfully merging this pull request may close these issues.

2 participants