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

[Dominik Csapak <d.csapak@proxmox.com>]

On 5/5/26 10:42 AM, Thomas Lamprecht wrote:
> 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.

understood

> 
> 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).

yeah, you're right, i'll make it part of the node config for pve/pbs
in a v3, thanks for the feedback!