fix(CSI-323): when snapshot of directory backed volumes is prohibited, incorrect error message is shown stating volume is legacy

sergeyberezansky · sergeyberezansky · commit da199a35fe97 · 2025-02-12T10:35:12.000+02:00
diff --git a/pkg/wekafs/volumeconstructors.go b/pkg/wekafs/volumeconstructors.go
@@ -279,7 +279,7 @@ func NewVolumeForCloneVolumeRequest(ctx context.Context, req *csi.CreateVolumeRe
 	}
 	if sourceVol.hasInnerPath() && !server.getConfig().allowSnapshotsOfLegacyVolumes {
 		// block cloning of snapshots from legacy volumes, as it wastes space
-		return nil, status.Errorf(codes.FailedPrecondition, "Cloning is not supported for Legacy CSI volumes")
+		return nil, status.Errorf(codes.FailedPrecondition, "Cloning is prohibited for directory-backed volumes, refer to WEKA CSI Plugin documentation for additional information")
 	}
 
 	// we assume the following when cloning a volume from existing volume:

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ func NewVolumeForCloneVolumeRequest(ctx context.Context, req *csi.CreateVolumeRe`
`279`	`279`	`}`
`280`	`280`	`if sourceVol.hasInnerPath() && !server.getConfig().allowSnapshotsOfLegacyVolumes {`
`281`	`281`	`// block cloning of snapshots from legacy volumes, as it wastes space`
`282`		`- return nil, status.Errorf(codes.FailedPrecondition, "Cloning is not supported for Legacy CSI volumes")`
	`282`	`+ return nil, status.Errorf(codes.FailedPrecondition, "Cloning is prohibited for directory-backed volumes, refer to WEKA CSI Plugin documentation for additional information")`
`283`	`283`	`}`
`284`	`284`
`285`	`285`	`// we assume the following when cloning a volume from existing volume:`