[pbs-devel] [PATCH proxmox-backup v7 6/8] api: add metricserver endpoints

Dominik Csapak d.csapak at proxmox.com
Wed Jun 8 10:23:10 CEST 2022


On 6/7/22 14:44, Thomas Lamprecht wrote:
> Am 02/06/2022 um 14:18 schrieb Dominik Csapak:
>> but in contrast to pve, we split the api by type of the section config,
>> since we cannot handle multiple types in the updater
>>
> 
> higher level comment:
> 
> I'd guess that it comes partially from how we named the perl module in PVE, but
> I'd still drop the "server" part from file/directory names and thus module/api
> path ones, IMO it doesn't really add much but extra length/noise (PVE's api also
> uses only "metrics")
> 

makes sense, in pve, we do '.../metrics/server' for the api though, should i do that?
or keep it just '.../metrics' ?







More information about the pbs-devel mailing list