New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete old database backups #328

Open
REBELinBLUE opened this Issue Mar 4, 2017 · 5 comments

Comments

2 participants
@REBELinBLUE
Owner

REBELinBLUE commented Mar 4, 2017

The updater backs up the database before running, but it never deletes old backups. Should be changed to only keep a few backups

@bbashy

This comment has been minimized.

Show comment
Hide comment
@bbashy

bbashy Mar 5, 2017

Contributor

Would this package not be better? https://github.com/spatie/laravel-backup

Can backup files as well as DB + clean them.

Contributor

bbashy commented Mar 5, 2017

Would this package not be better? https://github.com/spatie/laravel-backup

Can backup files as well as DB + clean them.

@REBELinBLUE

This comment has been minimized.

Show comment
Hide comment
@REBELinBLUE

REBELinBLUE Mar 5, 2017

Owner

It already uses that, just don't have it set up to purge old backups currently ;)

Owner

REBELinBLUE commented Mar 5, 2017

It already uses that, just don't have it set up to purge old backups currently ;)

@REBELinBLUE

This comment has been minimized.

Show comment
Hide comment
@REBELinBLUE

REBELinBLUE Mar 5, 2017

Owner

Tell a lie, currently using https://github.com/backup-manager/laravel I will have a look at that package

Owner

REBELinBLUE commented Mar 5, 2017

Tell a lie, currently using https://github.com/backup-manager/laravel I will have a look at that package

@bbashy

This comment has been minimized.

Show comment
Hide comment
@bbashy

bbashy Mar 5, 2017

Contributor

Yeah I checked to see what you used. Didn't see a purge/clean command for backup-manager?

Contributor

bbashy commented Mar 5, 2017

Yeah I checked to see what you used. Didn't see a purge/clean command for backup-manager?

@REBELinBLUE

This comment has been minimized.

Show comment
Hide comment
@REBELinBLUE

REBELinBLUE Mar 5, 2017

Owner

Yeah it doesn't seem to have it

Owner

REBELinBLUE commented Mar 5, 2017

Yeah it doesn't seem to have it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment