From 24f3f2bc2939680953aa61a9b2de3e1cec251a33 Mon Sep 17 00:00:00 2001
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
Date: Thu, 28 Apr 2022 19:39:52 +0200
Subject: [PATCH] ceph tools: make rados open if not reused shorter

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
 PVE/Ceph/Tools.pm | 25 +++++--------------------
 1 file changed, 5 insertions(+), 20 deletions(-)

diff --git a/PVE/Ceph/Tools.pm b/PVE/Ceph/Tools.pm
index 36d7788a..7291aa23 100644
--- a/PVE/Ceph/Tools.pm
+++ b/PVE/Ceph/Tools.pm
@@ -257,10 +257,7 @@ sub set_pool {
 
 sub create_pool {
     my ($pool, $param, $rados) = @_;
-
-    if (!defined($rados)) {
-	$rados = PVE::RADOS->new();
-    }
+    $rados = PVE::RADOS->new() if !defined($rados);
 
     my $pg_num = $param->{pg_num} || 128;
 
@@ -277,10 +274,7 @@ sub create_pool {
 
 sub ls_pools {
     my ($pool, $rados) = @_;
-
-    if (!defined($rados)) {
-	$rados = PVE::RADOS->new();
-    }
+    $rados = PVE::RADOS->new() if !defined($rados);
 
     my $res = $rados->mon_command({ prefix => "osd lspools" });
 
@@ -289,10 +283,7 @@ sub ls_pools {
 
 sub destroy_pool {
     my ($pool, $rados) = @_;
-
-    if (!defined($rados)) {
-	$rados = PVE::RADOS->new();
-    }
+    $rados = PVE::RADOS->new() if !defined($rados);
 
     # fixme: '--yes-i-really-really-mean-it'
     $rados->mon_command({
@@ -314,10 +305,7 @@ sub destroy_pool {
 #}]
 sub ls_fs {
     my ($rados) = @_;
-
-    if (!defined($rados)) {
-	$rados = PVE::RADOS->new();
-    }
+    $rados = PVE::RADOS->new() if !defined($rados);
 
     my $res = $rados->mon_command({ prefix => "fs ls" });
 
@@ -342,10 +330,7 @@ sub create_fs {
 
 sub destroy_fs {
     my ($fs, $rados) = @_;
-
-    if (!defined($rados)) {
-	$rados = PVE::RADOS->new();
-    }
+    $rados = PVE::RADOS->new() if !defined($rados);
 
     $rados->mon_command({
 	prefix => "fs rm",