[pve-devel] [PATCH ha-manager] get_pve_lock: allow retrying if pmxcfs is offline
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Nov 15 12:49:36 CET 2016
On 11/15/2016 12:08 PM, Dietmar Maurer wrote:
> # pve cluster filesystem not online
>> - die "can't create '$lockdir' (pmxcfs not mounted?)\n" if ! -d $lockdir;
>> + if (! -d $lockdir) {
>> + $self->log('err', "can't create '$lockdir' (pmxcfs not mounted?)");
>> + return 0;
>> + }
> You are inside 'eval', so this does not retry anything!
It actually did, I tested it before sending.
The method returns $got_lock at the end, which is 0 and so a retry happens.
But yes this is ugly and depends on side effects, I sent v2.
More information about the pve-devel
mailing list