You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
To solve this problem, and to ensure the ability to accommodate any filesystem / backup program in the future, we may want to extend the BackendBase abstract class, to add abstract methods such as optional_config_params and required_config_params. The backends could then control what parameters they accept in the user configuration. In the case of btrfs and rsync, they could have their configuration parameters set to src_dir, dst_dir, etc as they currently are. The zfs backend on the other hand could have its configuration parameters set to src_dataset, dst_dataset (optional), etc. This also opens the door to using more features of filesystems / backup tools such as backup compression or encryption if they are provided. One thing to keep in mind is that with this setup, the backend classes would also be responsible for providing parameter validation to the configuration parser.
The text was updated successfully, but these errors were encountered:
We should discuss this at our meeting tomorrow, as if we do decide that backend-specific configuration is the way to go then @SkiHatDuckie will be blocked on the configuration code until changes to the BackendBase class are implemented.
For some background please see this comment from PR #24.
To solve this problem, and to ensure the ability to accommodate any filesystem / backup program in the future, we may want to extend the
BackendBase
abstract class, to add abstract methods such asoptional_config_params
andrequired_config_params
. The backends could then control what parameters they accept in the user configuration. In the case of btrfs and rsync, they could have their configuration parameters set tosrc_dir
,dst_dir
, etc as they currently are. The zfs backend on the other hand could have its configuration parameters set tosrc_dataset
,dst_dataset
(optional), etc. This also opens the door to using more features of filesystems / backup tools such as backup compression or encryption if they are provided. One thing to keep in mind is that with this setup, the backend classes would also be responsible for providing parameter validation to the configuration parser.The text was updated successfully, but these errors were encountered: