On Wed, 21 Aug 2019 19:27:34 +0200 Rainer Gerhards
<rgerha...@hq.adiscon.com> wrote:
> I had a look but to me it seems we actually run out file handles. No
> failure indication.
>       
> Is this run in parallel? If so, how many tasks?
> 

I'm running the test-suite with the --no-parallel option, so no parallel
tests.
It seems to be the asynwr_deadlock2 test which fails.

I can reproduce this issue on a s390x porter machine by simply executing
the test

git bisect points at the following as the first faulty commit:

967b7eac17b601c773ee46017eed03cedf14be8d is the first bad commit
commit 967b7eac17b601c773ee46017eed03cedf14be8d
Merge: 2a33b2877 0b2477a9f
Author: Rainer Gerhards <rgerha...@adiscon.com>
Date:   Wed Jul 31 11:35:48 2019 +0200

    Merge pull request #3780 from rgerhards/i3760

    imrelp bugfix: hang after HUP

 plugins/imrelp/imrelp.c | 7 +++++--
 runtime/glbl.c          | 2 +-
 runtime/glbl.h          | 1 +
 3 files changed, 7 insertions(+), 3 deletions(-)

That doesn't look related immediately so I was wondering if git bisect
got a bit confused by the merge history.

So instead I tried reverting the following commit which was in the vicinity:

commit 7ad178cea388dd1ffc9ac17ae2fecae32c8443b3
Author: Rainer Gerhards <rgerha...@adiscon.com>
Date:   Wed Jul 31 11:36:27 2019 +0200

    omfile bugfix: race file when async writing is enabled

    This seems to be a long-standing bug, introduced around 7 years ago.
    It became more visible by properly closing files during HUP, which
    was done in 8.1905.0 (and was another bugfix).

    closes https://github.com/rsyslog/rsyslog/issues/3772

And tada, with that one reverted, asynwr_deadlock2 passes again on the
s390x porterbox.

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to