[pve-devel] [PATCH manager] ui: mpedit: activate backup on MP creation

[Aaron Lauterer]

On 11/28/19 2:26 PM, Thomas Lamprecht wrote:
> 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.

Yeah, I am not too sure myself. Let's see, we can test different 
variants. :)
> 
>> * 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.

Good point. Since the disks to be backed up is not a property of the job 
but the guests this should be okay in it's own view.

>