-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
Can be used for local backup? #10
Comments
If you want local backups you could also run a local MinIO instance (which is very lightweight when idling most of the time) that does nothing but serve as the target for your backup and you'd get all current features (including rotation) out of the box, plus your backups will also be stored locally. You can check the integration tests in this repository for an example of how this works. That being said, I am pretty sure the MinIO client could also rotate away files that are stored locally (as in no server) instead of remotely. Being honest, I am not super keen on adding this feature myself as I feel local backups defeat a very important reason for having backups, which is infrastructure failing, but I also see how it could be useful in some scenarios. If anyone wants to implement this I am happy to review and merge a PR that adds this feature. |
This is now possible as of |
Thanks for this, I tested it and works fine, it created the local backup correctly. I manually place an older file to test pruning, deletes file correcty but container log shows an error when deleting older files. This is my docker-compose config:
Tested on windows with wsl2 and throw this:
Tested with ubuntu server 20.04 with file owner different from root, delete files correctly but same message:
Tested setting filename to backup name, make it owned by root and same error. Deletes the file correctly but shows the same error. Beside that message, works perfectly. |
Hmm, it seems there are subtle differences between how |
I implemented a different way of rotating local backups (using |
Hi, i changed this: find $target* -delete -type f -mtime $BACKUP_RETENTION_DAYS to this: find $target* -type f -mtime $BACKUP_RETENTION_DAYS -exec rm -rf '{}' ';' Because the other way deletes everything and throws /archive is busy I submit a pull request with the change or if you can apply the change manually. |
I am not using kubernetes which MinIO seems to push. Can I still use it in the way you proposed above, together with this app? |
@lonix1 this issue is rather old and refers to an old version of the image. You can find extensive documentation on how.to do local backups using the image in the repository README. |
Wow you've been busy! So many new backup targets. Well done :) Any plans for services like dropbox / box ? |
Not from my end but I wouldn't object to merging a PR that adds it. If you feel like you want to add it yourself, I am happy to assist in getting it merged, else feel free to create a new issue requesting the feature and maybe someone else wants to pick it up. |
Thanks for the development of this feature, but I wonder if there is a way to make the backup locally? like /futurice/docker-volume-backup but keeping the file rotation.
The text was updated successfully, but these errors were encountered: