Implement secondary storage copy limit check for templates based on zone configuration

Author: Damans227

engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java

@@ -295,6 +295,22 @@ public void handleSysTemplateDownload(HypervisorType hostHyper, Long dcId) {
         }
     }
 
+    private boolean hasReachedSecStorageCopyLimit(VMTemplateVO template, long zoneId) {
+        boolean isPrivate = !template.isPublicTemplate() && !template.isFeatured()
+                && !TemplateType.SYSTEM.equals(template.getTemplateType());
+        int copyLimit = isPrivate
+                ? TemplateManager.PrivateTemplateSecStorageCopy.valueIn(zoneId)
+                : TemplateManager.PublicTemplateSecStorageCopy.valueIn(zoneId);
+        if (copyLimit <= 0) {
+            return false;
+        }
+        List<TemplateDataStoreVO> existing = _vmTemplateStoreDao.listByTemplateZoneDownloadStatus(
+                template.getId(), zoneId,
+                Status.DOWNLOADED, Status.DOWNLOAD_IN_PROGRESS, Status.NOT_DOWNLOADED);
+        int currentCopies = existing == null ? 0 : existing.size();
+        return currentCopies >= copyLimit;
+    }
+
     protected boolean shouldDownloadTemplateToStore(VMTemplateVO template, DataStore store) {
         Long zoneId = store.getScope().getScopeId();
         DataStore directedStore = _tmpltMgr.verifyHeuristicRulesForZone(template, zoneId);
@@ -304,6 +320,12 @@ protected boolean shouldDownloadTemplateToStore(VMTemplateVO template, DataStore
             return false;
         }
 
+        if (zoneId != null && hasReachedSecStorageCopyLimit(template, zoneId)) {
+            logger.info("Skipping sync of template [{}] to image store [{}]: zone [{}] has reached the configured copy limit.",
+                    template.getUniqueName(), store.getName(), zoneId);
+            return false;
+        }
+
         if (template.isPublicTemplate()) {
             logger.debug("Download of template [{}] to image store [{}] cannot be skipped, as it is public.", template.getUniqueName(),
                     store.getName());