[Bug]: Number of backups to keep not being respected #1257

Closed
opened 2026-02-20 01:16:56 -05:00 by deekerman · 3 comments
Owner

Originally created by @raviwarrier on GitHub (Jul 15, 2023).

Describe the issue

The old backups don't get deleted.
Screenshot 2023-07-16 083031
Screenshot 2023-07-16 083215

As you can see the setting is to keep 3 backup copies, but the app doesn't delete the older ones. Just noticed it in the debacle of 2.3.0 update and revert to 2.2.3.

Unfortunately, due to the migration to 2.3.0 and return to 2.2.3, all my old logs are no longer available.

Steps to reproduce the issue

Can't say.

Audiobookshelf version

2.2.3

How are you running audiobookshelf?

Docker

Originally created by @raviwarrier on GitHub (Jul 15, 2023). ### Describe the issue The old backups don't get deleted. ![Screenshot 2023-07-16 083031](https://github.com/advplyr/audiobookshelf/assets/42061703/ee679c88-d576-41fa-b87b-465ffac9951b) ![Screenshot 2023-07-16 083215](https://github.com/advplyr/audiobookshelf/assets/42061703/ff7eff6a-21d9-47e3-b6a3-4018d614ef02) As you can see the setting is to keep 3 backup copies, but the app doesn't delete the older ones. Just noticed it in the debacle of 2.3.0 update and revert to 2.2.3. Unfortunately, due to the migration to 2.3.0 and return to 2.2.3, all my old logs are no longer available. ### Steps to reproduce the issue Can't say. ### Audiobookshelf version 2.2.3 ### How are you running audiobookshelf? Docker
deekerman 2026-02-20 01:16:56 -05:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@advplyr commented on GitHub (Jul 20, 2023):

I don't think we should automatically delete old backups once you adjust the limit. It seems there was some bug happening where it kept more than 3 backups. If you can reproduce that bug let me know

@advplyr commented on GitHub (Jul 20, 2023): I don't think we should automatically delete old backups once you adjust the limit. It seems there was some bug happening where it kept more than 3 backups. If you can reproduce that bug let me know
Author
Owner

@raviwarrier commented on GitHub (Jul 21, 2023):

I will keep a look out for this in the new version 2.3.3. If it happens again, do you want me to send you the logs?

@raviwarrier commented on GitHub (Jul 21, 2023): I will keep a look out for this in the new version 2.3.3. If it happens again, do you want me to send you the logs?
Author
Owner

@advplyr commented on GitHub (Jul 21, 2023):

That would be great, just the logs around the time of the automated backup. It works by creating a new backup, then it will remove 1 backup if there are more than X amount of backups. Only 1 at a time is removed and this will be in the logs that it successfully removed 1 backup.

@advplyr commented on GitHub (Jul 21, 2023): That would be great, just the logs around the time of the automated backup. It works by creating a new backup, then it will remove 1 backup if there are more than X amount of backups. Only 1 at a time is removed and this will be in the logs that it successfully removed 1 backup.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/audiobookshelf#1257
No description provided.