[pve-devel] [PATCH manager] api: backup: remove unneeded call of rrd_dump

[pve-devel] [PATCH manager] api: backup: remove unneeded call of rrd_dump

[Aaron Lauterer]

$rrd is never used within this function, therefore we can easily remove
it.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
---
 PVE/API2/Backup.pm | 1 -
 1 file changed, 1 deletion(-)

diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm
index 42e9c418..27f5a14c 100644
--- a/PVE/API2/Backup.pm
+++ b/PVE/API2/Backup.pm
@@ -667,7 +667,6 @@ __PACKAGE__->register_method({
         my $vzconf = cfs_read_file('vzdump.cron');
         my $all_jobs = $vzconf->{jobs} || [];
         my $job;
-        my $rrd = PVE::Cluster::rrd_dump();
 
         for my $j (@$all_jobs) {
             if ($j->{id} eq $param->{id}) {
-- 
2.39.5



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

[pve-devel] applied: [PATCH manager] api: backup: remove unneeded call of rrd_dump

[Thomas Lamprecht]

On Tue, 01 Jul 2025 14:16:56 +0200, Aaron Lauterer wrote:
> $rrd is never used within this function, therefore we can easily remove
> it.
> 
> 

Applied, thanks!

[1/1] api: backup: remove unneeded call of rrd_dump
      commit: f454b458949e1d66b04eaa84d51128ac4ea82250


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