[pve-devel] [PATCH qemu-server 1/2] move checks to top

Dietmar Maurer dietmar at proxmox.com
Tue Jun 13 17:52:01 CEST 2017


> On 06/13/2017 05:11 PM, Dietmar Maurer wrote:
> > I think this change is wrong!
> >
> 
> any details as to why?

At least it changes the behavior of that method.
 
> we do not modify neither $attr nor $cleanup, so the data is the same in 
> both locations.
> 
> the only thing that could be different is if parse_volume_id or path 
> dies, or if $vtype is not 'images'

exactly (behaviour change).
 
> but if we mark a volume with 'replicate=0' this all should be irrelevant 
> anyway? else we fail with an error, instead of ignoring the marked disk...

Maybe. We really need regression tests for that.
 
> as for the cdrom, this fails already when you have one set to 'none' or 
> 'cdrom' so if this patch is wrong,
> there is still an error here

I would accept a patch for that ;-)




More information about the pve-devel mailing list