Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

List more archive metadata #8422

Merged

Conversation

ThomasWaldmann
Copy link
Member

@ThomasWaldmann ThomasWaldmann commented Sep 26, 2024

Looks like this now:

(borg-env) tw@MacBook-Pro borg % borg repo-list                    
60e521ba  Mon, 2024-09-23 22:57:11 +0200  docs             tw          MacBook-Pro  a comment
211b5d10  Thu, 2024-09-26 09:07:07 +0200  scripts          tw          MacBook-Pro
d7ceb3f1  Thu, 2024-09-26 10:05:23 +0200  scripts          tw          MacBook-Pro
bc8d1d7a  Thu, 2024-09-26 10:12:45 +0200  src              tw          MacBook-Pro
cee1c7e5  Thu, 2024-09-26 10:15:07 +0200  scripts          tw          MacBook-Pro
a07b7cd7  Thu, 2024-09-26 10:15:17 +0200  docs             tw          MacBook-Pro
(borg-env) tw@MacBook-Pro borg % borg repo-list --short
60e521ba5d44d6fed6aa984ded9301821e85d594971986097f96dc5b55597ef2
211b5d10d61558d8d85eb25c5f5c3c2f14c694c240f206c84bc007c39c4890cd
d7ceb3f14fdbd5142db18e03c93c29ca0269845118c9ab8f5b929dbd66c5fc82
bc8d1d7a814938f96c59e6b71b874fe95d478425e9ba3d77de9667dbb569d3e5
cee1c7e5e08d24339bff8ac96c08303179fa9a0268fe7a9315f9fdb68fca7e2e
a07b7cd759ab47a45ef43c9dfc62cef5eaa984995ebfeef6b0e75eac5888f90e

See the commit comments for the reasoning.

Needed to change this because listing just the
archive names is pretty useless if names are not
unique.

The short list is likely mostly used by scripts to
iterate over all archives, so outputting IDs is
better.
Copy link

codecov bot commented Sep 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.55%. Comparing base (628990d) to head (9b3a002).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8422      +/-   ##
==========================================
+ Coverage   81.54%   81.55%   +0.01%     
==========================================
  Files          70       70              
  Lines       12779    12779              
  Branches     2317     2317              
==========================================
+ Hits        10420    10422       +2     
+ Misses       1701     1700       -1     
+ Partials      658      657       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ThomasWaldmann ThomasWaldmann force-pushed the list-more-archive-metadata branch 2 times, most recently from 3c2a49d to a06c7dd Compare September 26, 2024 13:51
…adjust

borg 1.x encouraged users to put everything into the archive name:
- name of the dataset
- timestamp (usually used to make the archive name unique)
- maybe also hostname (when backing up to same repo from multiple hosts)
- maybe also username (when backing up to same repo from multiple users)

borg2 now discourages users from putting the timestamp into the name,
because we rather want same name within a series of archives - thus,
the field width for the name can be narrower.

the ID of the archive is now the only unique identifier, thus it is
moved to the leftmost place.
256bits (64 hex digits) was a bit much and as borg can also deal with
abbreviated IDs, we only show 32bits (8 hex digits) by default.

the ID is followed by the timestamp (also quite "interesting", because
it usually differs for different archives).

then following are: archive name, user name, host name - these might be
always the same if there is only one series of archives in a repo.

use 2 blanks separating the fields for better readability.
@ThomasWaldmann ThomasWaldmann marked this pull request as ready for review September 26, 2024 15:06
@ThomasWaldmann ThomasWaldmann merged commit 3c1f173 into borgbackup:master Sep 26, 2024
15 of 16 checks passed
@ThomasWaldmann ThomasWaldmann deleted the list-more-archive-metadata branch September 26, 2024 15:07
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.

1 participant