From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox Backup Server development discussion
<pbs-devel@lists.proxmox.com>,
Stefan Hanreich <s.hanreich@proxmox.com>
Subject: Re: [pbs-devel] [PATCH proxmox-backup v2 0/7] Add Prune Options to Sync Jobs
Date: Wed, 30 Nov 2022 17:23:23 +0100 [thread overview]
Message-ID: <2b9309af-052d-ee5f-a396-bfa0cff85ea3@proxmox.com> (raw)
In-Reply-To: <20221130150102.242374-1-s.hanreich@proxmox.com>
Am 30/11/2022 um 16:00 schrieb Stefan Hanreich:
> This patch adds the possibility of configuring prune options for sync jobs. This
> runs a prune job after a successful sync job that prunes the backup groups that
> got synced by the respective sync job (meaning this respects group-filter,
> max-depth, target namespace and so on).
Why? Ain't the existing prune job and their flexible schedules enough? Why
should one conflate syncing with pruning? Isn't that just complicating things
(many interfaces that can do everything of other interfaces).
Am 30/11/2022 um 16:00 schrieb Stefan Hanreich:
> Add KeepOptions parameters to pull & sync-job
> refactor prune.rs - add PruneJob struct for handling prune jobs
> Add pruning parameters to the pull command
> use new PruneJob in prune command
> use new PruneJob struct in Prune Jobs implementation
> add KeepOptions to Web UI of Sync Jobs
> Add documentation for prune options in Sync Jobs
having a lot of changelogs written, especially recently the lacks of human
readable tags (e.g., "docs:", "ui:" "sync job:", ...) and mixed casing stuck
quite out, even much more so the use of filenames in commit subjects which I
already pointed out quite directly and visible as being useless and annoying.
Structs are not always as worse, but not very often _that_ useful either for
skimming a bigger git log or assembling change somewhat meaningful logs.
next prev parent reply other threads:[~2022-11-30 16:23 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-30 15:00 Stefan Hanreich
2022-11-30 15:00 ` [pbs-devel] [PATCH proxmox-backup v2 1/7] Add KeepOptions parameters to pull & sync-job Stefan Hanreich
2022-11-30 15:00 ` [pbs-devel] [PATCH proxmox-backup v2 2/7] refactor prune.rs - add PruneJob struct for handling prune jobs Stefan Hanreich
2022-11-30 15:00 ` [pbs-devel] [PATCH proxmox-backup v2 3/7] Add pruning parameters to the pull command Stefan Hanreich
2022-11-30 15:00 ` [pbs-devel] [PATCH proxmox-backup v2 4/7] use new PruneJob in prune command Stefan Hanreich
2022-11-30 15:01 ` [pbs-devel] [PATCH proxmox-backup v2 5/7] use new PruneJob struct in Prune Jobs implementation Stefan Hanreich
2022-11-30 15:01 ` [pbs-devel] [PATCH proxmox-backup v2 6/7] add KeepOptions to Web UI of Sync Jobs Stefan Hanreich
2022-11-30 15:01 ` [pbs-devel] [PATCH proxmox-backup v2 7/7] Add documentation for prune options in " Stefan Hanreich
2022-11-30 16:23 ` Thomas Lamprecht [this message]
2022-12-01 9:50 ` [pbs-devel] [PATCH proxmox-backup v2 0/7] Add Prune Options to " Stefan Hanreich
2023-01-05 9:46 ` Thomas Lamprecht
2022-12-01 10:51 ` Lukas Wagner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2b9309af-052d-ee5f-a396-bfa0cff85ea3@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=pbs-devel@lists.proxmox.com \
--cc=s.hanreich@proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox