Skip to content

with-lock, info docs#2873

Merged
enkore merged 2 commits intoborgbackup:masterfrom
enkore:issue/2869
Jul 24, 2017
Merged

with-lock, info docs#2873
enkore merged 2 commits intoborgbackup:masterfrom
enkore:issue/2869

Conversation

@enkore
Copy link
Copy Markdown
Contributor

@enkore enkore commented Jul 24, 2017

Fixes #2869
Fixes #2868

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 24, 2017

Codecov Report

Merging #2873 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2873      +/-   ##
==========================================
- Coverage   86.26%   86.25%   -0.02%     
==========================================
  Files          23       23              
  Lines        8826     8826              
  Branches     1486     1486              
==========================================
- Hits         7614     7613       -1     
  Misses        820      820              
- Partials      392      393       +1
Impacted Files Coverage Δ
src/borg/archiver.py 87.54% <ø> (ø) ⬆️
src/borg/archive.py 82.88% <0%> (-0.08%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 836bc33...b4b58e7. Read the comment docs.

Number of files: {stats[nfiles]}
Command line: {command_line}
Utilization of max. archive size: {limits[max_archive_size]:.0%}
Utilization of maximum possible archive size: {limits[max_archive_size]:.0%}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/possible/supported/ ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that's more clear.

= all chunks in the repository.
= all chunks in the repository.

Borg only supports archives up to a certain size. The size of an archive
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, this is correct, but can be misunderstood by users who do not know that content data lives outside of the archive. so maybe rather say "... archive metadata streams up to ...".

Borg only supports archives up to a certain size. The size of an archive
relative to this limit depends on a number of factors, mainly the number
of files, the lengths of paths and other metadata stored for files.
This is shown as *utilization of maximum possible archive size*.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also update here if you change above.

If you copy a repository with the lock held, the lock will be present in
the copy, obviously. Thus, before using borg on the copy, you need to
use "borg break-lock" on it.
the copy. Thus, before using borg on the copy from or on a different host,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about what "from or on" shall mean here.

if you run the client "from" a machine, it means the same as if you run the client "on" a machine.

use "borg break-lock" on it.
the copy. Thus, before using borg on the copy from or on a different host,
you need to use "borg break-lock" on it.
(Borg is cautious and does not automatically remove stale locks made by a different host)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove parens, add dot?
or move it to before the final dot of previous sentence?

Copy link
Copy Markdown
Member

@ThomasWaldmann ThomasWaldmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@enkore enkore merged commit c1d7cd9 into borgbackup:master Jul 24, 2017
@enkore enkore deleted the issue/2869 branch July 24, 2017 21:50
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.

3 participants