[pve-devel] [PATCH cluster v4 3/4] cfs_lock: always include lockid in error

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Nov 9 09:47:26 CET 2017


Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 data/PVE/Cluster.pm | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm
index 1dd29b2..af30f09 100644
--- a/data/PVE/Cluster.pm
+++ b/data/PVE/Cluster.pm
@@ -868,17 +868,15 @@ my $cfs_lock = sub {
 
     my $filename = "$lockdir/$lockid";
 
-    my $msg = "can't aquire cfs lock '$lockid'";
-
     eval {
 
 	mkdir $lockdir;
 
 	if (! -d $lockdir) {
-	    die "$msg: pve cluster filesystem not online.\n";
+	    die "pve cluster filesystem not online.\n";
 	}
 
-	my $timeout_err = sub { die "$msg: got lock request timeout\n"; };
+	my $timeout_err = sub { die "got lock request timeout\n"; };
 	local $SIG{ALRM} = $timeout_err;
 
 	while (1) {
@@ -911,12 +909,12 @@ my $cfs_lock = sub {
 
     alarm(0);
 
-    $err = "$msg: no quorum!\n" if !$got_lock && !check_cfs_quorum(1);
+    $err = "no quorum!\n" if !$got_lock && !check_cfs_quorum(1);
 
     rmdir $filename if $got_lock; # if we held the lock always unlock again
 
     if ($err) {
-        $@ = $err;
+        $@ = "error with cfs lock '$lockid': $err";
         return undef;
     }
 
-- 
2.11.0





More information about the pve-devel mailing list