[pve-devel] applied: [PATCH many v4 00/31] Expand and migrate RRD data and add/change summary graphs

[Thomas Lamprecht]

Am 26.07.25 um 03:06 schrieb Aaron Lauterer:
> This patch series does a few things. It expands the RRD format for nodes and
>  VMs. For all types (nodes, VMs, storage) we adjust the aggregation to align
>  them with the way they are done on the Backup Server. Therefore, we have new
>  RRD defitions for all 3 types.


applied series with a bunch of follow-ups from Lukas (thanks!) and myself, thanks!

> KNOWN ISSUES:
> * on a live system, renaming the source RRD files to FILE.old doesn't seem to
> work as expected and besides the renamed ones, new ones without the .old prefix
> show up again. I suspect some interaction with rrdached and/or pmxcfs receiving
> new data.

FWIW, while I prepared a service to handle this on-boot, delaying that by more than
a one or two minutes is not ideal either, and while we could differentiate between
smaller and bigger setups (thus less or more time required for the migration), it
is probably simpler just doing it on the upgrade itself, albeit getting this
synchronized would be good too, maybe playing around with stopping rrdcached before
the upgrade might help, or make pmxcfs not commit anything if it sees a flag file
or the like, but it's only a short period of time of RRD metrics, nothing critical,
so I did not want to block this on that.

btw. It would be still very important to add a short section for this to the
upgrade guide, mentioning how one can continue the upgrade again later, not
all will see the message in the d/postinst output.