[pve-devel] [PATCH manager v3 6/7] improve tree/grid icons

[Dominik Csapak]

>
> I see they are two issues at stake here:
>  * storages like NFS, which are hanging the pvestatd in some corner
> cases, turning their owning nodes to the 'offline' status as no RRD data
> at all is delivered anymore.
> (for instance a stopped NFS server will hang pvestatd in -d $path test
> of PVE::Storage::Plugin::activate_storage(), as we noticed yesterday)

exactly why this series exists

>
>  * lack of visual status when a storage is offline in the GUI. Is
> checking the rrd status for this the best way ? We report via pvesm
> status inactive/offline storages which delivers the status faster that
> waiting for RRD data to be missing. Or I am missing something ?

we do not have anything else (cluster-wide) for updating the tree,
all other api calls to the storages are costly and can potentially hang

the 'status' which pvesm reports could be written into the rrd,
but then we would have to bump the version for storage rrd data

>
> A usability/UI question:
>  * when the status of a storage is marked as unknown/offline should the
> icon of the node where this storage is configured reflect this or not ?
> Currently it does not.

this was intended, as a 'faulty'/'misconfigured'/etc storage does not 
have to be inherently bad for the host (e.g. an iso storage)