[pve-devel] applied: [PATCH v2 manager] fix #3440: influxdb: remove duplicate vmid tag

Thomas Lamprecht t.lamprecht at proxmox.com
Wed May 26 17:39:31 CEST 2021


On 26.05.21 11:25, Lorenz Stechauner wrote:
> remove vmid from data part, it is already contained in object part.
> this is accomplished by adding the parameter $excluded to
> build_influxdb_payload().
> 
> Signed-off-by: Lorenz Stechauner <l.stechauner at proxmox.com>
> ---
> changes to v1:
> - not using `delete` anymore
> - added parameter `excluded` to build_influxdb_payload()
> 

nicer approach indeed.

>  PVE/Status/InfluxDB.pm | 13 ++++++++-----
>  1 file changed, 8 insertions(+), 5 deletions(-)
> 
>

applied, thanks! Followed up with a slight change in comment message, as
the old one read a bit to generic to me.





More information about the pve-devel mailing list