Skip to content

Commit 8ade4d4

Browse files
committed
fix for setting maxbackups on backup schedule
1 parent c1ff9ad commit 8ade4d4

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

engine/schema/src/main/java/org/apache/cloudstack/backup/BackupScheduleVO.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,12 +63,13 @@ public class BackupScheduleVO implements BackupSchedule {
6363
public BackupScheduleVO() {
6464
}
6565

66-
public BackupScheduleVO(Long vmId, DateUtil.IntervalType scheduleType, String schedule, String timezone, Date scheduledTimestamp) {
66+
public BackupScheduleVO(Long vmId, DateUtil.IntervalType scheduleType, String schedule, String timezone, Date scheduledTimestamp, Integer maxBackups) {
6767
this.vmId = vmId;
6868
this.scheduleType = (short) scheduleType.ordinal();
6969
this.schedule = schedule;
7070
this.timezone = timezone;
7171
this.scheduledTimestamp = scheduledTimestamp;
72+
this.maxBackups = maxBackups;
7273
}
7374

7475
@Override

server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -492,7 +492,7 @@ public BackupSchedule configureBackupSchedule(CreateBackupScheduleCmd cmd) {
492492

493493
final BackupScheduleVO schedule = backupScheduleDao.findByVMAndIntervalType(vmId, intervalType);
494494
if (schedule == null) {
495-
return backupScheduleDao.persist(new BackupScheduleVO(vmId, intervalType, scheduleString, timezoneId, nextDateTime));
495+
return backupScheduleDao.persist(new BackupScheduleVO(vmId, intervalType, scheduleString, timezoneId, nextDateTime, maxBackups));
496496
}
497497

498498
schedule.setScheduleType((short) intervalType.ordinal());

0 commit comments

Comments
 (0)