Skip to content

Update backup framework enabled config to global scope #11567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sureshanaparti wants to merge 8 commits into
base: 4.20
Choose a base branch
from
+10 −11
Open

Update backup framework enabled config to global scope #11567

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5d046c7
Check backup framework enabled config in zone scope first (before che…
sureshanaparti Sep 3, 2025
cc43873
Update backup.framework.enabled config to Global scope
sureshanaparti Sep 9, 2025
91d1a5c
Revert "Update backup.framework.enabled config to Global scope"
sureshanaparti Sep 23, 2025
a820333
Check the config key and name exists in its scope (without scopie id)
sureshanaparti Sep 24, 2025
af84896
code improvements
sureshanaparti Sep 25, 2025
0ed7291
Revert "code improvements"
sureshanaparti Jan 29, 2026
8d02dc0
Revert "Check the config key and name exists in its scope (without sc…
sureshanaparti Jan 29, 2026
e036a51
Revert "Revert "Update backup.framework.enabled config to Global scope""
sureshanaparti Jan 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion api/src/main/java/org/apache/cloudstack/backup/BackupManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public interface BackupManager extends BackupService, Configurable, PluggableSer
ConfigKey<Boolean> 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<String> BackupProviderPlugin = new ConfigKey<>("Advanced", String.class,
"backup.framework.provider.plugin",
Expand Down
19 changes: 9 additions & 10 deletions server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -947,13 +947,13 @@ public boolean configure(String name, Map<String, Object> 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");
}
}

Expand Down Expand Up @@ -1128,7 +1128,7 @@ public void scheduleBackups() {
continue;
}

if (isDisabled(vm.getDataCenterId())) {
if (isDisabled()) {
continue;
}

Expand Down Expand Up @@ -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);
Expand Down
Loading