Support for --verify-data #210

Closed
opened 2019-09-01 03:20:19 +00:00 by adatum · 7 comments

Would you consider adding support for borg check's --verify-data option, as a config file option (eg. in the consistency section) and/or command line option (eg. borgmatic check --verify-data)?

Would you consider adding support for `borg check`'s `--verify-data` option, as a config file option (eg. in the consistency section) and/or command line option (eg. `borgmatic check --verify-data`)?
Owner

Sure! Sounds like it's even slower than a normal archive check, but sometimes that may be exactly the sort of validation you need. Given that it's similar in spirit to the existing Borg consistency checks, I would probably support specifying this within the configuration's checks option, just like with the existing repository, archives or extract.

Sure! Sounds like it's even slower than a normal archive check, but sometimes that may be exactly the sort of validation you need. Given that it's similar in spirit to the existing Borg consistency checks, I would probably support specifying this within the configuration's `checks` option, just like with the existing `repository`, `archives` or `extract`.
Author

Thanks! Since --verify-data seems to be an order of magnitude slower than check without it, right now I don't think I will schedule it but rather run it manually on occasion. So, a command line option would be great for that. Though if it's too much trouble to implement, I understand.

Thanks! Since `--verify-data` seems to be an order of magnitude slower than `check` without it, right now I don't think I will schedule it but rather run it manually on occasion. So, a command line option would be great for that. Though if it's too much trouble to implement, I understand.
Owner

Not so much trouble as it opens up a bunch of questions about the other consistency checks: Should there be a corresponding --check-archives command-line option? What about --check-repository? --check-extract?

Not so much trouble as it opens up a bunch of questions about the other consistency checks: Should there be a corresponding `--check-archives` command-line option? What about `--check-repository`? `--check-extract`?
Author

I certainly won't complain about more options. The main use-case in my mind for command line options for check is to manually invoke tasks that take a long time to run, and therefore might not be scheduled to run automatically. --verify-data and possibly --check-extract seem to fall in that category.

I certainly won't complain about more options. The main use-case in my mind for command line options for check is to manually invoke tasks that take a long time to run, and therefore might not be scheduled to run automatically. `--verify-data` and possibly `--check-extract` seem to fall in that category.
Owner

I implemented the new "data" consistency check in master. Next, I'll look into adding something to the borgmatic command-line to suit your use case.

I implemented the new "data" consistency check in master. Next, I'll look into adding something to the borgmatic command-line to suit your use case.
Owner

Okay, this is now implemented and released (in 1.3.16) via a new borgmatic check --only flag. For instance, you can use borgmatic check --only data. And it can be specified multiple times.

Let me know how it works out for you!

Okay, this is now implemented and released (in 1.3.16) via a new `borgmatic check --only` flag. For instance, you can use `borgmatic check --only data`. And it can be specified multiple times. Let me know how it works out for you!
Owner

However, note this bug with extract checks: witten/borgmatic#217

However, note this bug with `extract` checks: https://projects.torsion.org/witten/borgmatic/issues/217
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#210
No description provided.