From f9a982b6e3b9520d0f4dba49f1fab67f93403671 Mon Sep 17 00:00:00 2001
From: Harikrishna Patnala <harikrishna.patnala@gmail.com>
Date: Thu, 22 Jan 2026 18:49:36 +0530
Subject: [PATCH] Removed unused code

---
 .../orchestration/StorageOrchestrator.java    | 30 -------------------
 .../storage/image/TemplateServiceImpl.java    |  3 +-
 2 files changed, 2 insertions(+), 31 deletions(-)

diff --git a/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/StorageOrchestrator.java b/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/StorageOrchestrator.java
index 93832468c4a..933b4e0c5ce 100644
--- a/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/StorageOrchestrator.java
+++ b/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/StorageOrchestrator.java
@@ -638,36 +638,6 @@ public class StorageOrchestrator extends ManagerBase implements StorageOrchestra
         }
     }
 
-    private class CopyTemplateTask implements Callable<TemplateApiResult> {
-        private TemplateInfo sourceTmpl;
-        private DataStore destStore;
-        private String logid;
-
-        public CopyTemplateTask(TemplateInfo sourceTmpl, DataStore destStore) {
-            this.sourceTmpl = sourceTmpl;
-            this.destStore = destStore;
-            this.logid = ThreadContext.get(LOGCONTEXTID);
-        }
-
-        @Override
-        public TemplateApiResult call() {
-            ThreadContext.put(LOGCONTEXTID, logid);
-            TemplateApiResult result;
-            AsyncCallFuture<TemplateApiResult> future = templateService.copyTemplateToImageStore(sourceTmpl, destStore);
-            try {
-                result = future.get();
-            } catch (ExecutionException | InterruptedException e) {
-                logger.warn("Exception while copying template [{}] from image store [{}] to image store [{}]: {}",
-                        sourceTmpl.getUniqueName(), sourceTmpl.getDataStore().getName(), destStore.getName(), e.toString());
-                result = new TemplateApiResult(sourceTmpl);
-                result.setResult(e.getMessage());
-            }
-            tryCleaningUpExecutor(destStore.getScope().getScopeId());
-            ThreadContext.clearAll();
-            return result;
-        }
-    }
-
     private class CopyTemplateFromSecondaryStorageTask implements Callable<TemplateApiResult> {
         private final long srcTemplateId;
         private final DataStore destStore;
diff --git a/engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java b/engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
index ce31e853d78..31f9c947778 100644
--- a/engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
+++ b/engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
@@ -69,6 +69,7 @@ import org.apache.cloudstack.storage.datastore.db.TemplateDataStoreVO;
 import org.apache.cloudstack.storage.image.datastore.ImageStoreEntity;
 import org.apache.cloudstack.storage.image.store.TemplateObject;
 import org.apache.cloudstack.storage.to.TemplateObjectTO;
+import org.apache.commons.collections.CollectionUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.logging.log4j.Logger;
 import org.apache.logging.log4j.LogManager;
@@ -646,7 +647,7 @@ public class TemplateServiceImpl implements TemplateService {
             List<DataStore> storesInOtherZone = _storeMgr.getImageStoresByZoneIds(otherZoneId);
             logger.debug("Checking zone [{}] for template [{}]...", otherZoneId, tmplt.getUniqueName());
 
-            if (storesInOtherZone == null || storesInOtherZone.isEmpty()) {
+            if (CollectionUtils.isEmpty(storesInOtherZone)) {
                 logger.debug("Zone [{}] has no image stores. Skipping.", otherZoneId);
                 continue;
             }