[pve-devel] [PATCH manager 2/2] gui: BackupView: improve filtering of backups

[Dominik Csapak]

On 2/21/20 4:33 PM, Thomas Lamprecht wrote:
> Am 2/21/20 um 11:02 AM schrieb Dominik Csapak:
>> use 'gridfilter' plugin to filter by vmid (returned from the api),
>> but leave the 'search' textbox (so that the user can still search
>> for e.g. a year)
>>
> 
> what's the improvement the subject talks about? It's way more hidden now
> how to also show other VM/CT, one needs to guess right that the slightly-bold
> and italics rendering of "VMID" means filter applied, click on that, click on
> filters and then edit that, not sure how much of an improvements this is..

the improvement is that we filter with info from the api instead of the 
naming of files. this will help us when we have plugins that do not
format the names of backups with 'vzdump-qemu-ID-...' (and would not 
work there)

as for the filtering itself, we already use that kind of filters
in the 'bulk' windows, so it is not unprecedented, though
i agree we could make it a little more obvious with e.g. css
(more contrast, maybe an icon?)

the other alternative (that i wanted to avoid), is to have
some 'magic' syntax in the search field to only search for vmid
(and not for the year e.g.)

> 
> Also this only works if VMID storage patch is not available, which you just
> cannot guarantee. If we, why ever, really want to do this I need a fallback
> here.
> 
i guess you mean 'is available', and yes thats the point (the api 
documents the vmid since 2018 as 'optional') but i am not sure
how we could change that in a good and backwards compatible way...
(filtering by volid is not really the right thing if we have
the vmid defined in the api)