Skip to content
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

Support rclone #6

Merged
merged 3 commits into from
Oct 19, 2019
Merged

Support rclone #6

merged 3 commits into from
Oct 19, 2019

Conversation

maximbaz
Copy link
Contributor

@maximbaz maximbaz commented Oct 2, 2019

Because I have to use FTP (yeah...)

rclone has to be present in the image, and config needs to be mounted to actually use it.

Tested (with lines 4-5 of backup commented out because of #4), works well 🙂

Copy link
Owner

@djmaze djmaze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nitpicks, otherwise a good idea.

@maximbaz
Copy link
Contributor Author

Done 👍

I had to move away from restic sadly (I couldn't get go-cron to trigger for some reason, but more importantly my provider has a limit on the number of files created on FTP and restic exceeds that limit), but I did apply your suggestions, hopefully this will be useful to someone else.

@djmaze
Copy link
Owner

djmaze commented Oct 11, 2019

Okay.. So is anyone else still interested in this? @cyrinux, you gave a thumbsup?

Otherwise, I would rather close this for now.

@maximbaz
Copy link
Contributor Author

@cyrinux and I both have the same FTP limitations; if you don't want this functionality then I'm closing 👍

@maximbaz maximbaz closed this Oct 11, 2019
@cyrinux
Copy link

cyrinux commented Oct 12, 2019 via email

@maximbaz
Copy link
Contributor Author

Ah ok, in that case reopening and addressing the final comments

@maximbaz maximbaz reopened this Oct 12, 2019
@djmaze djmaze merged commit a0046ad into djmaze:master Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants