Re: [PATCH datacenter-manager v2 1/4] lib/api/ui: add location property to remote config

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

Am 05.05.26 um 10:35 schrieb Dominik Csapak:
> 
> 
> On 5/5/26 10:24 AM, Thomas Lamprecht wrote:
>> Am 05.05.26 um 09:31 schrieb Dominik Csapak:
>>> this will be used to show the remote on a map in a custom view.
>>> Let's the user simply enter the longitude and latitude in the remote
>>> edit window.
>>
>> Would be nicer and more fitting to have this on the remote side, i.e. the
>> node configs as metadata or similar format-string field.
>>
>> And one location per remote won't cut it for PVE. We know of clusters that
>> are distributed over different location with a few km distance in between
>> and a dark fiber channel for near LAN latency to upheld cluster communication
>> requirements.
> 
> do you mean that the user configures that on the pve and pbs side per node?

exactly what I meant.
> if yes, sure that is doable. I didn't go this way to not exclude
> users with older clusters (e.g. 8.4) to use that feature.
> also it's metadata that the pve nodes themselves won't really use

If you really want to provide those old releases this features we can port
it back, that said, EOL is nearing for those (bit over ~3 months left), and
especially for this non-essential feature I'd just not bother with that.

And why should we not show this on the PVE nodes themselves? And independent
of that it's in anyway a property of a PVE/ node and tied to that physical
location, not a property of a PDM remote config entry, which should IMO not
care about these things, at least not as primary source of such information
(overrides can be OK, but IMO not really justified here).