From f3cafb98d00814e619ae415e42ef0d55699aa359 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Erki=20Ma=CC=88rks?= Date: Mon, 11 May 2026 15:06:01 +0300 Subject: [PATCH] Fix HypervisorType dispatch in addFullClone flag helper Replace invalid switch on Hypervisor.HypervisorType (not a Java enum) with equality checks so cloud-engine-storage-datamotion compiles. --- .../storage/motion/AncientDataMotionStrategy.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java b/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java index c5654e13db1..89c9e699ec5 100644 --- a/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java +++ b/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java @@ -280,14 +280,14 @@ public class AncientDataMotionStrategy implements DataMotionStrategy { if (dataTO == null) { return dataTO; } - switch (dataTO.getHypervisorType()) { - case VMware: - return addFullCloneAndDiskprovisiongStrictnessFlagOnVMwareDest(dataTO); - case XenServer: - return addFullCloneAndDiskprovisiongStrictnessFlagOnXenServerDest(dataTO); - default: - return dataTO; + Hypervisor.HypervisorType hypervisorType = dataTO.getHypervisorType(); + if (Hypervisor.HypervisorType.VMware.equals(hypervisorType)) { + return addFullCloneAndDiskprovisiongStrictnessFlagOnVMwareDest(dataTO); } + if (Hypervisor.HypervisorType.XenServer.equals(hypervisorType)) { + return addFullCloneAndDiskprovisiongStrictnessFlagOnXenServerDest(dataTO); + } + return dataTO; } protected Answer copyObject(DataObject srcData, DataObject destData) {