diff --git a/api/src/main/java/org/apache/cloudstack/backup/BackupManager.java b/api/src/main/java/org/apache/cloudstack/backup/BackupManager.java index 78d189c3bf1d..7aa0176f1a01 100644 --- a/api/src/main/java/org/apache/cloudstack/backup/BackupManager.java +++ b/api/src/main/java/org/apache/cloudstack/backup/BackupManager.java @@ -40,7 +40,7 @@ public interface BackupManager extends BackupService, Configurable, PluggableSer ConfigKey BackupFrameworkEnabled = new ConfigKey<>("Advanced", Boolean.class, "backup.framework.enabled", "false", - "Is backup and recovery framework enabled.", false, ConfigKey.Scope.Zone); + "Is backup and recovery framework enabled.", false); ConfigKey BackupProviderPlugin = new ConfigKey<>("Advanced", String.class, "backup.framework.provider.plugin", diff --git a/server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java b/server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java index dc2677a507f0..134b81489b6f 100644 --- a/server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java +++ b/server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java @@ -947,13 +947,13 @@ public boolean configure(String name, Map params) throws Configu return true; } - public boolean isDisabled(final Long zoneId) { - return !(BackupFrameworkEnabled.value() && BackupFrameworkEnabled.valueIn(zoneId)); + public boolean isDisabled() { + return !(BackupFrameworkEnabled.value()); } private void validateForZone(final Long zoneId) { - if (zoneId == null || isDisabled(zoneId)) { - throw new CloudRuntimeException("Backup and Recovery feature is disabled for the zone"); + if (zoneId == null || isDisabled()) { + throw new CloudRuntimeException("Backup and Recovery feature is disabled"); } } @@ -1128,7 +1128,7 @@ public void scheduleBackups() { continue; } - if (isDisabled(vm.getDataCenterId())) { + if (isDisabled()) { continue; } @@ -1237,12 +1237,11 @@ protected void runInContext() { if (logger.isTraceEnabled()) { logger.trace("Backup sync background task is running..."); } + if (isDisabled()) { + logger.debug("Backup Sync Task is not enabled. Skipping!"); + return; + } for (final DataCenter dataCenter : dataCenterDao.listAllZones()) { - if (dataCenter == null || isDisabled(dataCenter.getId())) { - logger.debug("Backup Sync Task is not enabled in zone [{}]. Skipping this zone!", dataCenter == null ? "NULL Zone!" : dataCenter); - continue; - } - final BackupProvider backupProvider = getBackupProvider(dataCenter.getId()); if (backupProvider == null) { logger.warn("Backup provider not available or configured for zone {}", dataCenter);