remove noload for SharedFS read-only mounts

Paul Kommu · Paul Kommu · commit 02b4b2c07108 · 2025-12-18T20:31:53.000Z
diff --git a/internal/node/fs/fs_node.go b/internal/node/fs/fs_node.go
@@ -69,8 +69,7 @@ func (d *Node) NodePublishVolume(ctx context.Context, request *csi.NodePublishVo
 
 	if request.GetReadonly() {
 		// Read-only volumes cannot be written to in any way
-		// We should not attempt to replay the journal
-		mountOpts = append(mountOpts, node.ReadOnlyMountOption, node.NoLoadMountOption)
+		mountOpts = append(mountOpts, node.ReadOnlyMountOption)
 	}
 
 	err = nodePublishVolume(d.Mounter, d.Resizer, mountOpts, nfsEnabled, d.NFSRemotePorts, d.NFSHost, request)

Original file line number	Diff line number	Diff line change
`@@ -69,8 +69,7 @@ func (d Node) NodePublishVolume(ctx context.Context, request csi.NodePublishVo`
`69`	`69`
`70`	`70`	`if request.GetReadonly() {`
`71`	`71`	`// Read-only volumes cannot be written to in any way`
`72`		`- // We should not attempt to replay the journal`
`73`		`- mountOpts = append(mountOpts, node.ReadOnlyMountOption, node.NoLoadMountOption)`
	`72`	`+ mountOpts = append(mountOpts, node.ReadOnlyMountOption)`
`74`	`73`	`}`
`75`	`74`
`76`	`75`	`err = nodePublishVolume(d.Mounter, d.Resizer, mountOpts, nfsEnabled, d.NFSRemotePorts, d.NFSHost, request)`