[pve-devel] [PATCH manager 0/4] followups for vzdump scheduling2

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Nov 10 18:18:00 CET 2021


On 10.11.21 15:02, Dominik Csapak wrote:
> * fixes the saving/loading of the 'prune-backups' field (oops)
> * adds correct validation for id
> * adds a comment field for jobs

still missing any documentation update ;)

I'd move out the schedule format section from the replication chapter to an appendix
(or some other, more central place), extend backups about the new scheduling stuff.

also the ID is now some in-between stuff, one needs to enter it but it won't be
shown in the grid anyway, so either:

1 go the full way like PBS sync/verify/... jobs and just auto-generate, less inputs
  or the user is normally better UX and they can express comments already in the,
  well new comment field.
2 hide it only for the old ones, rather too confusing
3 allow to edit it for existing jobs, would be a bit weird for the API but as we have
  the digest mechanism and can  do (pseudo call) `PUT /.../backup/<oldid>?id=<newid>`
  it could work without to much confusion..

1 would be the least commitment now, 2 is meh and 3 is more commitment now and may
open a precedent for users to nag about allowing this in more places (where it may
be way harder, a nuisance or plain impossible)

What do you think?





More information about the pve-devel mailing list