Don't delete backup when we dissociate backup offering

This commit is contained in:
Pierre Charton 2024-11-15 11:13:26 +01:00
parent 4e0ea82bcb
commit bd0bc0f39d
2 changed files with 2 additions and 29 deletions

View File

@ -168,34 +168,7 @@ public class BackrollBackupProvider extends AdapterBase implements BackupProvide
@Override
public boolean removeVMFromBackupOffering(VirtualMachine vm) {
logger.info("Removing VM ID {} from Backrool backup offering ", vm.getUuid());
boolean isAnyProblemWhileRemovingBackups = false;
List<Backup> backupsInCs = backupDao.listByVmId(null, vm.getId());
for (Backup backup : backupsInCs) {
logger.debug("Trying to remove backup with id {}", backup.getId());
try {
if (getClient(backup.getZoneId()).deleteBackup(vm.getUuid(), getBackupName(backup))) {
logger.info("Backup {} deleted in Backroll for virtual machine {}", backup.getId(), vm.getName());
if (!backupDao.remove(backup.getId())){
isAnyProblemWhileRemovingBackups = true;
}
logger.info("Backup {} deleted in CS for virtual machine {}", backup.getId(), vm.getName());
} else {
isAnyProblemWhileRemovingBackups = false;
}
} catch (KeyManagementException | NoSuchAlgorithmException | IOException e) {
throw new CloudRuntimeException("Failed to remove VM from backup offering");
}
}
if (isAnyProblemWhileRemovingBackups) {
logger.info("Problems occured while removing some backups for virtual machine {}", vm.getName());
}
return isAnyProblemWhileRemovingBackups;
return true;
}
@Override

View File

@ -424,7 +424,7 @@ const user = {
const ldapEnable = (response.ldapconfigurationresponse.count > 0)
commit('SET_LDAP', ldapEnable)
}).catch(error => {
reject(error)
console.error(error)
})
api('cloudianIsEnabled').then(response => {