Re: [PATCH datacenter-manager/proxmox-geojson-data/yew-widget-toolkit/yew-widget-toolkit-assets v3 00/11] add a new map widget for custom views

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 22.05.26 um 10:34 schrieb Dominik Csapak:
> Locations are now taken from PVE/PBS themselves and are cached.
> I chose 24 hours as default max-age for the cache for now, but we might
> want to increase that to something even higher (maybe a month?).
> There is also currently no mechanism that updates the location in the background
> automatically, but this can be done in the next version or as a followup
> if we can agree on a sensible interval for that. (I'd suggest once a week
> or something like that?)

I mean, that data is in fast to parse "hot" configs tracked by pmxcfs, so
I'd not bother to much with an overly long caching policy, even "just" an
hour would be fine here, especially as we can also pull other data from
those configs while at it, at least for PVE (descriptions, acme domains,
...?). OTOH, this is probably something where one wants like 10s or less
if one actually is in process to set that up initially and after that,
weeks. months or just opportunistically (if that config needs to be
fetched for something else) in between would be fine.

> If there is a differing opinion or suggestion on any of that, please do tell.

Fine for the MVP for now. A force-refresh for a remote or all remotes is
basically the only thing needed that gives admin an escape hatch. As IMO,
one normally detects that it's wrong when checking that map, so one fixes
the data and rechecks the map, if it then needs 24h or 1h or 1 week doesn't
matter, as that's all to slow for that operation. With a force reload we
can make the period of the reload matter way less.

Anyway, I'll try to look at this in detail today, starting out with
proxmox-geojson-data package.