On Wed, Nov 20, 2019 at 09:09:12AM +0100, Fabian Ebner wrote:
> On 11/19/19 12:05 PM, Tim Marx wrote:
> > The bugfix for #2317 introduced a kind of odd api behavior, where each 
> > volume

While Tim's patch does indeed fix the API, I agree with Fabian's objections:

> Isn't it necessary to do this filtering inside 'list_volumes' and
> probably even for each volume?
> Otherwise for a storage with content types rootdir and images:
> 'pvesm list local-lvm --content rootdir' doesn't list anything and
> 'pvesm list local-lvm --content images' lists both container rootdirs
> and VM images.
> 

Why does `pvesm list local-iso --content images` actually list container data?
According to our wiki content type images is "KVM-Qemu VM images" which 
containers are not.
So currently the set of container images is a subset of VM images and there is 
no (easy) way to get the set VM images minus container images?

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

Reply via email to