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

contrib: added the common mixins library #346

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jkellerer
Copy link
Collaborator

@jkellerer jkellerer commented Mar 19, 2024

This PR is a contribution of a common mixins library that may be used to simplify common actions when performing backups. It starts with support for snapshots and databases in posix environments.

Status:

  • integration tested
  • tested

Status for the moment, it is just a brain dump of something I wanted to add for some time but it is highly untested, though I use most of the actions in my backups in simplified forms.

@creativeprojects
Copy link
Owner

creativeprojects commented Mar 19, 2024

This is an excellent idea 💡

I always wanted to add a section (in the documentation) on how to use rclone with restic with a few configuration examples, but never took the time to do it 😞

@creativeprojects
Copy link
Owner

BTW if we want the various snippets to also work with Windows, we can do like go-task and use a shell interpreter

This is out of the scope of this MR though

@jkellerer
Copy link
Collaborator Author

Was actually thinking to extend the shell hook format with an optional struct to toggle things like:

  • what shell to use
  • async (run until the section ends, e.g. for remote locks with external tools)
  • hidden-execution (copy to script file to hide confidentials from proc list)
  • what to do on return code (abort, fail, continue).. allows to use it as conditional when set to abort instead of fail.

...also out of scope for this PR. But helpful here and an integration point for a shell interpreter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants