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

--stats output should include repo location #5491

Closed
laur89 opened this issue Nov 8, 2020 · 7 comments · Fixed by #5570
Closed

--stats output should include repo location #5491

laur89 opened this issue Nov 8, 2020 · 7 comments · Fixed by #5570

Comments

@laur89
Copy link

laur89 commented Nov 8, 2020

Have you checked borgbackup docs, FAQ, and open Github issues?

Y

Is this a BUG / ISSUE report or a QUESTION?

Feature request

System information. For client/server mode post info for both machines.

Your borg version (borg -V).

borg 1.1.14 (from apk community repos)

Operating system (distribution) and version.

alpine linux

Full borg commandline that lead to the problem (leave away excludes and passwords)

borg create --stats

Describe the problem you're observing.

I'd like --stats output also to include the repository location. This is to make it easier to discern different --stats outputs from a logfile. I have a script that's backing up in parallel to remote & local borg repos; It'd be nice for --stats output head to look like

------------------------------------------------------------------------------
Repository: [email protected]:/repo/path   # for remote repo
Repository: /path/to/local/repo                                # for local repo
Archive name: gitea-downtime-2020-11-08-054300
Archive fingerprint: 27992ecef61844cb5ee377d447f2984bd9fc4e62134dcf5d81706914fg97pqfe
Time (start): Sun, 2020-11-08 05:43:08
@ThomasWaldmann ThomasWaldmann added this to the 1.1.15 milestone Nov 8, 2020
@ThomasWaldmann
Copy link
Member

Would you like to make a PR for that against master branch?

@ThomasWaldmann ThomasWaldmann changed the title [FR] --stats output to include repo location --stats output should include repo location Nov 8, 2020
@laur89
Copy link
Author

laur89 commented Nov 10, 2020

Not familiar with the codebase. I'll try to get around to it, but no promises.

@Gu1nness
Copy link
Contributor

@laur89 If you have some difficulties, I can have a look at it and point you to the right places :)

@Gu1nness
Copy link
Contributor

Gu1nness commented Nov 23, 2020

@laur89 Any news, do you need some help?
If you prefer, I can develop it and explain to you how the whole thing works.

@laur89
Copy link
Author

laur89 commented Nov 25, 2020

@Gu1nness local dev env setup turned out to be trickier than expected. Hoping to revisit coming weekend.

@Gu1nness
Copy link
Contributor

Ok nice, don't hesitate to ping me if necessary.

@Gu1nness
Copy link
Contributor

Okay @laur89 I'll patch it if you don't mind.

ThomasWaldmann added a commit that referenced this issue Dec 16, 2020
create: add repository location to --stats output, fixes #5491
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants