Add redbot-setup restore cli command#6709
Add redbot-setup restore cli command#6709Jackenmen wants to merge 32 commits intoCog-Creators:V3/developfrom
redbot-setup restore cli command#6709Conversation
|
In regards to the docs on this, the current backup and restore page mentions (the possible need of) fixing permissions on Linux with the
|
There should be no need for this unless the user has a seriously broken system. That's not on us to help with, and users will definitely just try running this command if something breaks for them due to unrelated reasons.
Yes, I missed that earlier and have just noticed in testing that we are, in fact, not checking whether the path is accessible at any point. I just fixed that.
The updated code should make it so this fails on the |
|
Okay, I added support for non-interactive use, and this is now (again) ready for review. Please note, however, that as described in the description, this PR depends on #6696 to work correctly, and the changeset from that PR is not included in this PR's branch. |
While nothing prevents future versions from adding more files to the backup, it seems simpler to get ahead of it...
|
I've made a slight change to metadata stored in the backup file - instead of a Another small change is the addition of |
Description of the changes
Fixes #2947, depends on #6696, supersedes #3681
I added progress bars to indicate to the user that the command does something but they're surely not the most accurate progress bars - progress bar for (un)packing archives are based on file amount, not on file size and progress bar for downloading repos is based on repo amount. I'm open to improving it but I think it still serves its purpose even if it's not very linear.
Backup can now have a version of its own in case we need to change something about its format later.
Docs: https://red-discordbot--6709.org.readthedocs.build/en/6709/backup_red.html
Have the changes in this PR been tested?
Yes