[pbs-devel] [PATCH proxmox/proxmox-backup v4] add metrics server capability

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Feb 1 12:39:33 CET 2022


On 01.02.22 12:01, Matthias Heiserer wrote:
> The patch generally works (connecting with influxdb server, sending data,
> using api to create config), but some issues/suggestions:
> 

Thanks for testing, would you consider providing a
> Tested-by: Matthias Heiserer <m.heiserer at proxmox.com>

tag even though below notes?

> Passing a value to the --enable in api create parameter always results in an
> error: "Error: parameter verification errors parameter 'enable': Expected
> boolean value."

`1` and `0` too?

> 
> Some personal preferences: I'd prefer if "api get" showed whether a token was
> set in the config, instead of leaving the field blank.

Hard to do "inline" as tokens are free-form, the cleanest way would to report
an extra boolean property on GET ("token-set").

> When adding a metricserver via "api create" using an already existing id, the
> old server gets overwritten. This might be undesired/problematic. Potential
> fix would be to forbid creating metric servers with an existing id.
> 

Yeah that violates the POST-only-once principle we try to adhere and must not
happen.





More information about the pbs-devel mailing list