Re: [pve-devel] [PATCH container 1/1] status: rrddata: use fixed pve-vm-9.0 path

["Laurențiu Leahu-Vlăducu" <l.leahu-vladucu@proxmox.com>]

This patch needs a (very small) rebase to apply correctly.


Reviewed-by: Laurențiu Leahu-Vlăducu <l.leahu-vladucu@proxmox.com>
Tested-by: Laurențiu Leahu-Vlăducu <l.leahu-vladucu@proxmox.com>


On 05.09.25 15:55, Aaron Lauterer wrote:
> Because we now always create it, should it not exists and old data is
> used from the old files.
> 
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
>   src/PVE/API2/LXC.pm | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm
> index 1f89c87..0abe705 100644
> --- a/src/PVE/API2/LXC.pm
> +++ b/src/PVE/API2/LXC.pm
> @@ -718,9 +718,9 @@ __PACKAGE__->register_method({
>       code => sub {
>           my ($param) = @_;
>   
> -        my $path = "pve-vm-9.0/$param->{vmid}";
> -        $path = "pve2-vm/$param->{vmid}" if !-e "/var/lib/rrdcached/db/${path}";
> -        return PVE::RRD::create_rrd_graph($path, $param->{timeframe}, $param->{ds}, $param->{cf});
> +        return PVE::RRD::create_rrd_graph(
> +            "pve-vm-9.0/$param->{vmid}", $param->{timeframe}, $param->{ds}, $param->{cf},
> +        );
>   
>       },
>   });
> @@ -762,9 +762,9 @@ __PACKAGE__->register_method({
>       code => sub {
>           my ($param) = @_;
>   
> -        my $path = "pve-vm-9.0/$param->{vmid}";
> -        $path = "pve2-vm/$param->{vmid}" if !-e "/var/lib/rrdcached/db/${path}";
> -        return PVE::RRD::create_rrd_data($path, $param->{timeframe}, $param->{cf});
> +        return PVE::RRD::create_rrd_data(
> +            "pve-vm-9.0/$param->{vmid}", $param->{timeframe}, $param->{cf},
> +        );
>       },
>   });
>   



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel