From 0a4ab936763ed789f8040d8109f4207346b6db62 Mon Sep 17 00:00:00 2001 From: carlory Date: Wed, 30 Apr 2025 17:07:43 +0800 Subject: [PATCH] fix incorrect comments --- pkg/kubelet/volumemanager/cache/desired_state_of_world.go | 4 ++-- pkg/volume/plugins.go | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/kubelet/volumemanager/cache/desired_state_of_world.go b/pkg/kubelet/volumemanager/cache/desired_state_of_world.go index e9bdb05aaed..afde30bf7f7 100644 --- a/pkg/kubelet/volumemanager/cache/desired_state_of_world.go +++ b/pkg/kubelet/volumemanager/cache/desired_state_of_world.go @@ -296,8 +296,8 @@ func (dsw *desiredStateOfWorld) AddPodToVolume( err) } } else { - // For non-attachable and non-device-mountable volumes, generate a unique name based on the pod - // namespace and name and the name of the volume within the pod. + // For non-attachable and non-device-mountable volumes, generate a unique name based on the unique pod + // name (refer to podUID) and the name of the volume within the pod. volumeName = util.GetUniqueVolumeNameFromSpecWithPod(podName, volumePlugin, volumeSpec) } diff --git a/pkg/volume/plugins.go b/pkg/volume/plugins.go index c4e5fee2851..632f0df017a 100644 --- a/pkg/volume/plugins.go +++ b/pkg/volume/plugins.go @@ -174,8 +174,6 @@ type VolumePlugin interface { ConstructVolumeSpec(volumeName, volumePath string) (ReconstructedVolume, error) // SupportsMountOption returns true if volume plugins supports Mount options - // Specifying mount options in a volume plugin that doesn't support - // user specified mount options will result in error creating persistent volumes SupportsMountOption() bool // SupportsSELinuxContextMount returns true if volume plugins supports