Merge pull request #2234 from k8s-infra-cherrypick-robot/cherry-pick-2231-to-release-1.25

[release-1.25] fix: skip chmod setting in readonly mode

Author: k8s-ci-robot

pkg/blob/nodeserver.go

@@ -389,8 +389,12 @@ func (d *Driver) NodeStageVolume(ctx context.Context, req *csi.NodeStageVolumeRe
 		}
 
 		if performChmodOp {
-			if err := chmodIfPermissionMismatch(targetPath, os.FileMode(mountPermissions)); err != nil {
-				return nil, status.Error(codes.Internal, err.Error())
+			if !isReadOnlyFromCapability(volumeCapability) {
+				if err := chmodIfPermissionMismatch(targetPath, os.FileMode(mountPermissions)); err != nil {
+					return nil, status.Error(codes.Internal, err.Error())
+				}
+			} else {
+				klog.V(2).Infof("skip chmod on targetPath(%s) since it's a read-only mount", targetPath)
 			}
 		} else {
 			klog.V(2).Infof("skip chmod on targetPath(%s) since mountPermissions is set as 0", targetPath)

test/e2e/dynamic_provisioning_test.go

@@ -270,6 +270,7 @@ var _ = ginkgo.Describe("[blob-csi-e2e] Dynamic Provisioning", func() {
 						VolumeMount: testsuites.VolumeMountDetails{
 							NameGenerate:      "test-volume-",
 							MountPathGenerate: "/mnt/test-",
+							ReadOnly:          true,
 						},
 						AccessModes: []v1.PersistentVolumeAccessMode{v1.ReadOnlyMany},
 					},