On 11/28/19 2:16 PM, Aaron Lauterer wrote:
> I do like the idea of seeing in the backup jobs panel which parts get backed 
> up :)
> 
> So my proposal on how to proceed is as follows:
> 
> * Send a v2 of this patch with better naming. Unless we come up with 
> something better "Include in backup".

OK.

> * align the VM disk panel to the LXC MP panel:
>   - rename checkbox and behavior to include, not exclude from backup

OK.

>   - enable by default on new disks

OK.

>   - move from advanced area to normal area?

Hmm, maybe? I mean no hard feelings here.

> * enhance backup job view to include a detailed list of disks per guest which 
> shows the status, included, not included, not possible to back up (bind 
> mounts)

I'd maybe try to add this in it's own "view-only" window with a own button in
the top-bar of DC/Backup, the job editor is already complex enough IMO.


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to