repo commands renamed r* -> repo-*

[ThomasWaldmann]

Fixes #8292.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 97.67442% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.74%. Comparing base (3794e32) to head (cfd35b4).
Report is 5 commits behind head on master.

Files with missing lines	Patch %	Lines
src/borg/archiver/repo_compress_cmd.py	83.33%	1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8358   +/-   ##
=======================================
  Coverage   81.73%   81.74%           
=======================================
  Files          70       70           
  Lines       12648    12648           
  Branches     2287     2287           
=======================================
+ Hits        10338    10339    +1     
  Misses       1665     1665           
+ Partials      645      644    -1     

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

[ThomasWaldmann]

While working on this, I thought we maybe should rename some other commands also.

disambiguation / split vs. describing scope

The initial reason to have * and repo-* commands was to split and disambiguate them, e.g. delete deletes archive(s), while repo-delete deletes a repository. This makes the argparser and docs a lot easier, if 2 rather different commands are not mixed into 1 (as it was in borg 1.x).

But the command name could also describe the scope the command works on: full repo vs. archive(s).

compact

Thus compact maybe should be rather repo-compact (same as we already have repo-compress although there was no disambiguation needed for that).

check

This is currently a mixed-scope command, there is the --repository-only part doing the full repo low-level check and the --archives-only higher level check. By default it runs both of them.

prune

Difficult. Potentially works on the full repo, but rules rather apply on the archive level.

[ThomasWaldmann]

@RonnyPfannschmidt @witten @m3nu @sophie-h ^

[RonnyPfannschmidt]

I'd not give prune/check prefixes for now

The repo only/Archive only versions could be turned into separate prefixed commands

Prune is kind of a repo wide operation that's specifically targeting the full list of archives

[ThomasWaldmann]

Still need feedback on borg compact vs. borg repo-compact.

That works on full repo scope, but it's doing the garbage collection by looking at all archives.

[ThomasWaldmann]

Will merge this soon to avoid conflicts. Can work on compact in another PR...

[m3nu]

Still need feedback on borg compact vs. borg repo-compact.

When in doubt and since this operates on both, I'd keep the old version of borg compact.

[witten]

I second keeping borg compact as-is, unless you're planning on splitting it into two separate subcommands, one for repos and one for archives.

[ThomasWaldmann]

@witten there is nothing to compact "in archives".

It just needs to analyze the archives to know what chunks are in use, so it can delete those which aren't used. It also computes some stats as a side effect (-v).