From 501ec15e402efea47e0fcca38e0fb77a45cb3c7c Mon Sep 17 00:00:00 2001 From: IvanHunters Date: Mon, 15 Jun 2026 12:41:00 +0300 Subject: [PATCH] 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 --- packages/system/kubevirt-cdi/templates/cdi-cr.yaml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/system/kubevirt-cdi/templates/cdi-cr.yaml b/packages/system/kubevirt-cdi/templates/cdi-cr.yaml index 7dc15ee0a..82d2793d0 100644 --- a/packages/system/kubevirt-cdi/templates/cdi-cr.yaml +++ b/packages/system/kubevirt-cdi/templates/cdi-cr.yaml @@ -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 . }}