@@ -508,7 +508,6 @@ local_backup(){
508508 backups_count=$( echo " $backup_list " | wc -l)
509509 if [ " $BACKUPS " -le " $backups_count " ]; then
510510 backups_rm_number=$(( backups_count - BACKUPS))
511- (( ++ backups_rm_number))
512511
513512 # Removing old backup
514513 for backup in $( echo " $backup_list " | head -n $backups_rm_number ) ; do
@@ -624,7 +623,7 @@ ftp_backup() {
624623 fi
625624 backups_count=$( echo " $backup_list " | wc -l)
626625 if [ " $backups_count " -ge " $BACKUPS " ]; then
627- backups_rm_number=$(( backups_count - BACKUPS + 1 ))
626+ backups_rm_number=$(( backups_count - BACKUPS))
628627 for backup in $( echo " $backup_list " | head -n $backups_rm_number ) ; do
629628 backup_date=$( echo $backup | sed -e " s/$user .//" -e " s/.tar$//" )
630629 echo -e " $( date " +%F %T" ) Rotated ftp backup: $backup_date " | \
@@ -779,7 +778,7 @@ sftp_backup() {
779778 fi
780779 backups_count=$( echo " $backup_list " | wc -l)
781780 if [ " $backups_count " -ge " $BACKUPS " ]; then
782- backups_rm_number=$(( backups_count - BACKUPS + 1 ))
781+ backups_rm_number=$(( backups_count - BACKUPS))
783782 for backup in $( echo " $backup_list " | head -n $backups_rm_number ) ; do
784783 backup_date=$( echo $backup | sed -e " s/$user .//" -e " s/.tar.*$//" )
785784 echo -e " $( date " +%F %T" ) Rotated sftp backup: $backup_date " | \
@@ -828,7 +827,7 @@ google_backup() {
828827 backup_list=$( ${gsutil} ls gs://$BUCKET /$BPATH /$user .* 2> /dev/null)
829828 backups_count=$( echo " $backup_list " | wc -l)
830829 if [ " $backups_count " -ge " $BACKUPS " ]; then
831- backups_rm_number=$(( backups_count - BACKUPS + 1 ))
830+ backups_rm_number=$(( backups_count - BACKUPS))
832831 for backup in $( echo " $backup_list " | head -n $backups_rm_number ) ; do
833832 echo -e " $( date " +%F %T" ) Roated gcp backup: $backup "
834833 $gsutil rm $backup > /dev/null 2>&1
0 commit comments