From 22bc1399631e51cc968e8a640fb903af6b9fce44 Mon Sep 17 00:00:00 2001 From: chrislusf Date: Thu, 11 Dec 2025 10:55:46 -0800 Subject: fix: clarify that mount service is required for node component Update comments and validation message to specify that the mount service is required for the CSI driver 'node component', not the entire driver. This is more accurate for controller-only deployments. --- deploy/helm/seaweedfs-csi-driver/templates/daemonset.yaml | 2 +- deploy/helm/seaweedfs-csi-driver/values.yaml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/deploy/helm/seaweedfs-csi-driver/templates/daemonset.yaml b/deploy/helm/seaweedfs-csi-driver/templates/daemonset.yaml index ad1979f..ee5bc7d 100644 --- a/deploy/helm/seaweedfs-csi-driver/templates/daemonset.yaml +++ b/deploy/helm/seaweedfs-csi-driver/templates/daemonset.yaml @@ -1,5 +1,5 @@ {{- if and .Values.node.enabled (not .Values.mountService.enabled) -}} -{{- fail "mountService.enabled must be true when node.enabled is true. The CSI driver requires the mount service to function." -}} +{{- fail "mountService.enabled must be true when node.enabled is true. The CSI driver node component requires the mount service to mount volumes." -}} {{- end -}} {{- if .Values.node.enabled}} {{- $mountEndpoint := default .Values.mountService.endpoint .Values.node.mountEndpoint }} diff --git a/deploy/helm/seaweedfs-csi-driver/values.yaml b/deploy/helm/seaweedfs-csi-driver/values.yaml index 46320d7..edcc631 100644 --- a/deploy/helm/seaweedfs-csi-driver/values.yaml +++ b/deploy/helm/seaweedfs-csi-driver/values.yaml @@ -73,12 +73,12 @@ seaweedfsCsiPlugin: # Mount Service Configuration # The mount service runs as a separate DaemonSet that manages FUSE mounts. -# This is required for the CSI driver to function properly. +# This is required for the CSI driver node component to function properly. # The node.mountEndpoint, node.mountSocketDir, and node.mountHostPath values # default to the corresponding mountService.* values when left empty. # NOTE: This deploys an additional DaemonSet with privileged containers. mountService: - # Must be enabled for the CSI driver to mount volumes + # Must be enabled for the CSI driver node component to mount volumes enabled: true image: chrislusf/seaweedfs-mount:dev # Endpoint for communication between CSI driver and mount service -- cgit v1.2.3