On 7/8/25 13:09, Lukas Wagner wrote:
> Something that I have noticed is that when you have a second disk/mountpoint 
> on
> some storage, detach the disk/mountpoint, remove the storage and then
> try to remove the unused disk, there is a difference in behavior between
> containers and VMs. For VMs, the ununused disk can be removed, while for 
> containers
> the removal fails with:
> 
>   unused0: unable to apply pending change unused0 : storage 'test' does not 
> exist 
> 

Sorry for the late reply and thanks for having a look at this! Good
catch. Considering some of the other patches in this series are aimed at
streamlining the behavior of VMs and CTs, I'll add a check similar to
the one done for VMs in a v9.

> 
> Also did a brief review of the code. I'm not really familiar with these parts
> of our stack, but from my point of view these changes look fine.
> 
> 
> Tested-by: Lukas Wagner <l.wag...@proxmox.com>
> Reviewed-by: Lukas Wagner <l.wag...@proxmox.com>
> 
> 



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

Reply via email to