Seems the "keep" is still commented out in latest dev. 8.x-4.x-dev
Is this something that we should expect soon? Any problem on this? Because currently the backup cumulate infinitely.
Thanks

/src/Form/ScheduleForm.php line 91

//    $form['keep'] = array(
//      '#type' => 'textfield',
//      '#title' => $this->t('Number to keep'),
//      '#default_value' => $backup_migrate_schedule->get('keep'),
//      '#description' => $this->t('The number of backups to retain. Once this number is reached, the oldest backup will be deleted to make room for the most recent backup. Leave blank to keep all backups.'),
//      '#size' => 10,
//    );
CommentFileSizeAuthor
#4 keep-backups.jpg20.05 KBcouturier
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MastaP created an issue. See original summary.

couturier’s picture

Duplicate issue Delete old backups on schedule has been marked.

MastaP’s picture

Right. All duplicates, yet none solve the problem or even gets addressed. ;)

couturier’s picture

Title: $backup_migrate_schedule->get('keep') » Add field to "Schedule" for number of backups to keep
FileSize
20.05 KB

In the UI, Backup and Migrate 8.x-4.x-alpha2 has a title field showing number of backups to keep (see screenshot attached), but when you go to "Add Schedule" or "Edit" a schedule, there is no menu or field to specify the number of backups to keep. It appears that this was meant to be a part of the UI, but currently users can work around it in the code.

  • ronan committed d725f92 on 8.x-4.x
    Issue #2862618 by couturier: Add field to "Schedule" for number of...
ronan’s picture

Status: Active » Fixed

Should be there now. Previously created backups may have to be hand deleted since previously B&M was not correctly saving the schedule id in the file metadata. That means schedules don't know they own older backups and can't delete them.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.