implement remote-ratelimit #45

New Issue

import_bot · 2018-01-05T05:34:44Z

import_bot commented

2018-01-05 05:34:44 +00:00

Adding support for borgs --remote-ratelimit parameter would be perfect.

Regular backups should not disrupt normal operations by using all bandwidth. Using a ratelimit would be the way to go.

Imported from Taiga issue 44 (done). Created on 2017-11-01T15:44:38+0000 by Benjamin Häublein.

Adding support for borgs `--remote-ratelimit` parameter would be perfect. Regular backups should not disrupt normal operations by using all bandwidth. Using a ratelimit would be the way to go. --- Imported from Taiga issue 44 (done). Created on 2017-11-01T15:44:38+0000 by Benjamin Häublein.

import_bot closed this issue

2018-01-05 05:34:44 +00:00

import_bot commented

2018-01-05 05:34:45 +00:00

Just released borgmatic with support for a "remote_rate_limit" option in the "storage" section of config. Let me know how it works out for you.

Comment on 2017-11-03T05:40:14+0000 by Dan Helfman.

Just released borgmatic with support for a "remote_rate_limit" option in the "storage" section of config. Let me know how it works out for you. --- Comment on 2017-11-03T05:40:14+0000 by Dan Helfman.

import_bot commented

2018-01-05 05:34:45 +00:00

I get the following error when using the remote_rate_limit option:

validation.invalid
 --- All found errors ---
["Key 'remote_rate_limit' was not defined. Path: '/storage'"]
Errors found but will not raise exception...
An error occurred while parsing a configuration file at documents.yaml:
Key 'remote_rate_limit' was not defined. Path: '/storage'

Comment on 2017-11-04T09:54:40+0000 by Benjamin Häublein.

I get the following error when using the `remote_rate_limit` option: ``` validation.invalid --- All found errors --- ["Key 'remote_rate_limit' was not defined. Path: '/storage'"] Errors found but will not raise exception... An error occurred while parsing a configuration file at documents.yaml: Key 'remote_rate_limit' was not defined. Path: '/storage' ``` --- Comment on 2017-11-04T09:54:40+0000 by Benjamin Häublein.

import_bot commented

2018-01-05 05:34:46 +00:00

Huh, that configuration option totally works here. (Just confirmed again.) Would you mind pasting your borgmatic configuration file? Please feel free to obscure any sensitive information (passphrases, hostnames, etc).

Comment on 2017-11-04T18:23:47+0000 by Dan Helfman.

Huh, that configuration option totally works here. (Just confirmed again.) Would you mind pasting your borgmatic configuration file? Please feel free to obscure any sensitive information (passphrases, hostnames, etc). --- Comment on 2017-11-04T18:23:47+0000 by Dan Helfman.

import_bot commented

2018-01-05 05:34:47 +00:00

Also, please double check that you're using the new version of borgmatic that contains this change! It's borgmatic 1.1.10 in particular. You can run pip3 show borgmatic to see that version number.

Comment on 2017-11-04T18:24:59+0000 by Dan Helfman.

Also, please double check that you're using the new version of borgmatic that contains this change! It's borgmatic 1.1.10 in particular. You can run `pip3 show borgmatic` to see that version number. --- Comment on 2017-11-04T18:24:59+0000 by Dan Helfman.

import_bot commented

2018-01-05 05:34:47 +00:00

Shame on me. I only upgraded using pip. Even checked the new versions code on my machine. Totally forgot about python3..

Thanks. Works perfectly.

Comment on 2017-11-05T13:09:54+0000 by Benjamin Häublein.

**Shame on me. I only upgraded using pip. Even checked the new versions code on my machine. Totally forgot about python3..** **Thanks. Works perfectly.** --- Comment on 2017-11-05T13:09:54+0000 by Benjamin Häublein.

import_bot commented

2018-01-05 05:34:47 +00:00

Glad to hear it's working!

Comment on 2017-11-05T17:46:40+0000 by Dan Helfman.

Glad to hear it's working! --- Comment on 2017-11-05T17:46:40+0000 by Dan Helfman.

Sign in to join this conversation.