fix(kubevirt-cdi): drop global cloneStrategyOverride

The CDI CR pinned cloneStrategyOverride to csi-clone globally
(introduced with the golden-disks feature, switched from snapshot to
csi-clone afterwards). That override bypassed CDI's same-StorageClass
guard, so a cross-StorageClass clone (e.g. a vm-disk created from a
golden image in a replicated StorageClass into a node-local one) was
issued as csi-clone and could never satisfy on the target driver: the
temporary clone PVC stayed unbound forever and no error surfaced.

Drop the override and let CDI pick the clone strategy per
StorageProfile. The same-class path still goes through CSI-snapshot
clones (fast); the cross-class path falls back to host-assisted copy
(slow but correct).

Closes #2908

Signed-off-by: IvanHunters <xorokhotnikov@gmail.com>
This commit is contained in:
IvanHunters committed 2026-06-15 12:41:00 +03:00
1 parent 42ca3e6bd3
commit 501ec15e40
1 file changed
+8 -1
@@ -3,7 +3,14 @@ kind: CDI
metadata:
name: cdi
spec:
cloneStrategyOverride: csi-clone
# No cloneStrategyOverride: let CDI pick the clone strategy per StorageProfile.
# The previous global "csi-clone" override bypassed CDI's same-StorageClass
# guard, so cross-StorageClass clones (e.g. cloning a golden image from a
# replicated StorageClass into a node-local one) were issued as csi-clone
# and could never satisfy on the target driver — the PVC stayed unbound.
# Per-StorageProfile selection lets CSI-snapshot clones stay fast for the
# same-class case while falling back to host-assisted copy across classes.
# See https://github.com/cozystack/cozystack/issues/2908
config:
{{- with .Values.uploadProxyURL }}
uploadProxyURLOverride: {{ quote . }}