[pve-devel] [PATCH series 0/4] Add GUI for disk reassignment

[Aaron Lauterer]

Thx for the review and the problems found.


On 8/31/20 1:04 PM, Dominic Jäger wrote:
> I reassigned some disks and it seemed to work nice for me :)
> Error messages about unavailable disks for running VMs and equal source and
> target VM made sense to me, correct possible targets were displayed, multiple
> storages worked.
> 
> One thing that came to my mind: It might make sense to display what name the
> reassigned disk got in the target VM in the log.
> For example "Reassigned to VM 123 as Unused Disk 7 [maybe with volid]".
> Because if there are multiple unused disks on the target VM it could happen
> that it's not the one with the highest number?


Sounds good. I will incorporate that in the other patch series that does deal with the backend.


> 
> On Tue, Aug 25, 2020 at 05:14:22PM +0200, Aaron Lauterer wrote:
>> This patch series adds the GUI to the recent patch series [0] which
>> enables the reassignment of disks between VMs.
>>
>> For this to work, the previous patch series [0] needs to be applied and
>> installed.