ovh/the-bastion
1
1
Fork 0
mirror of https://github.com/ovh/the-bastion.git synced 2025-10-25 04:47:30 +08:00
Code Issues Projects Releases Packages Wiki Activity

fix: groupdel: false positive in lock contention detection

Browse source 
Groups that were containing 'lock' or 'retry' in their name
would falsely trigger the /etc/passwd and /etc/group lock
contention detection, due to their presence in the output of
the system command, implying several retries that were not
needed.
This commit is contained in:
Stéphane Lesimple 2021-09-24 10:13:40 +00:00 • committed by Stéphane Lesimple
parent b40a2fd6e3
commit a6488ee6fb
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
lib/perl/OVH/Bastion/os.inc
View file

@ -47,9 +47,14 @@ sub _sys_autoretry {
my $fnret;
foreach my $try (1 .. 10) {
$fnret = OVH::Bastion::execute(%params);
if ( ($fnret->value && $fnret->value->{'sysret'} == 10)
|| ($fnret->value && $fnret->value->{'stdout'} && grep { /retry|lock/i } @{$fnret->value->{'stdout'}})
|| ($fnret->value && $fnret->value->{'stderr'} && grep { /retry|lock/i } @{$fnret->value->{'stderr'}}))
if (
($fnret->value && $fnret->value->{'sysret'} == 10)
|| (
($fnret->value && $fnret->value->{'sysret'} != 0)
&& ( ($fnret->value->{'stdout'} && grep { /retry|lock/i } @{$fnret->value->{'stdout'}})
|| ($fnret->value->{'stderr'} && grep { /retry|lock/i } @{$fnret->value->{'stderr'}}))
)
)
{
# too much concurrency, sleep a bit and retry
warn_syslog('Too much concurrency on try '