From: Dominik Csapak <d.csapak@proxmox.com>
Date: Mon, 15 Nov 2021 13:57:19 +0000 (+0100)
Subject: api: metrics: use complete config for testing the connection
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=5964deeae33ed61006e37d13c1bc022b6a220727;p=pve-manager.git

api: metrics: use complete config for testing the connection

not only the given parameters, e.g. at the moment, the gui will
never send a 'verify-certificate' parameter, even if set in the config

by using the complete resulting config, we test the actual settings.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---

diff --git a/PVE/API2/Cluster/MetricServer.pm b/PVE/API2/Cluster/MetricServer.pm
index a6b99d4f..882cda10 100644
--- a/PVE/API2/Cluster/MetricServer.pm
+++ b/PVE/API2/Cluster/MetricServer.pm
@@ -238,7 +238,7 @@ __PACKAGE__->register_method ({
 
 	    $plugin->on_update_hook($id, $data, $sensitive_params);
 
-	    $plugin->test_connection($opts, $id);
+	    $plugin->test_connection($data, $id);
 
 	    PVE::Cluster::cfs_write_file('status.cfg', $cfg);
 	});