[pve-devel] [PATCH manager] api: metrics: use complete config for testing the connection

Dominik Csapak d.csapak at proxmox.com
Mon Nov 15 14:57:19 CET 2021


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 at proxmox.com>
---
 PVE/API2/Cluster/MetricServer.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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);
 	});
-- 
2.30.2






More information about the pve-devel mailing list