Dan Helfman witten
Loading Heatmap…

witten commented on issue witten/borgmatic#121

Missing source_dir = fatal error

That's correct!

1 week ago

witten opened issue witten/borgmatic#136

Exceptions raised during argument parsing are unhandled

1 week ago

witten commented on issue witten/borgmatic#135

Get erros / Could not find the error

Oh, and `exclude_patterns` needs to go in the `location:` section, so that'll have to be moved. You can check out the original config file schema here for reference: https://projects.torsion.org/witten/borgmatic/src/branch/master/borgmatic/config/schema.yaml

1 week ago

witten commented on issue witten/borgmatic#135

Get erros / Could not find the error

It looks like you're missing the `storage:` section header. Judging by your config file, it should probably go above `encryption_passphrase`. The `storage:` header should have the same indent level as the existing `location:` header.

1 week ago

witten commented on issue witten/borgmatic#121

Missing source_dir = fatal error

If you wanted to try to reproduce this, you could add an intentional error into one of your configuration files so as to induce an error. For instance, changing the repository to a path that doesn't exist.

1 week ago

witten closed issue witten/borgmatic#134

META: when we answer on an issue/PR we are not subscribed auto

1 week ago

witten commented on issue witten/borgmatic#134

META: when we answer on an issue/PR we are not subscribed auto

This is a current Gitea limitation, mentioned in this open Gitea ticket: https://github.com/go-gitea/gitea/issues/145

1 week ago

witten closed issue witten/borgmatic#133

Error while parsing a configuration file: Key 'compression' was not defined. Path: '/location'

1 week ago

witten commented on issue witten/borgmatic#133

Error while parsing a configuration file: Key 'compression' was not defined. Path: '/location'

I think @sdellenb is correct here. While the error message could be more helpful, it's basically the validator's way of saying "Uh, there's this key `compression` in the `location` section, and I don't expect a `compression` key there."

1 week ago

witten commented on issue witten/borgmatic#132

Exclude failing due to confusing expansion of *

Thanks for reporting this. In borgmatic source, it looks like the function that expands `*` and `~` in paths is called when interpreting both `source_directories` and `exclude_patterns`. So I think the fix would be to continue expanding `*` for `source_directories` but not for `exclude_patterns`. And `exclude_patterns` still needs to expand `~` so as not to regress #58.

2 weeks ago

witten commented on issue witten/borgmatic#131

Cron + borgmatic + "set -e" = unknown error

Got it. Hope that works!

2 weeks ago

witten commented on issue witten/borgmatic#131

Cron + borgmatic + "set -e" = unknown error

I'm curious.. What was it?

2 weeks ago

witten pushed to master at witten/provisioning

2 weeks ago

witten commented on issue witten/borgmatic#130

Do you keep your passphrase in the config file?

Correct, at least not in my case.

2 weeks ago

witten commented on issue witten/borgmatic#107

Encapsulate subprocess calls in run() function

For anyone else looking at this ticket, here's the WIP PR: https://projects.torsion.org/witten/borgmatic/pulls/109

2 weeks ago

witten commented on issue witten/borgmatic#109

WIP Replace subprocess calls with a generic run function

Please let me know if there's any way I can help with this one! It would be great to get it merged at some point.

2 weeks ago

witten commented on issue witten/borgmatic#129

"before_backup" option has 2 bugs

Glad to hear that cleared it up! Yet another reason to do #53 soon!

2 weeks ago

witten commented on issue witten/borgmatic#127

Small suggestion for default "echo date"

Yeah, I think the ultimate solution is to implement real logging as per #53. That way we'd get full timestamp configuration support of whatever logging solution is used. And then perhaps even removing the `date` portion of these example echos.

2 weeks ago