On Tue, Nov 08, 2016 at 06:03:04PM +0100, Andreas Schwab wrote:
> Commit 5babb5bdb3 ("t6026-merge-attr: clean up background process at end
> of test case") added a kill command to clean up after the test, but this
> can fail if the sleep command exits before the cleanup is executed.
> Ignore the error from the kill command.
>
> Signed-off-by: Andreas Schwab <sch...@suse.de>
> ---
> The failure can be simulated by adding a sleep after the last command to
> delay the cleanup.
Thanks for the reproduction hint. I sometimes run the test suite through
a "stress" script that sees if a test script racily fails under load,
but I wasn't able to trigger it here (I guess a full second is just too
long even under high load). But the extra sleep makes it obvious.
Looks like the original is in v2.10.1, so this should probably go to
maint, as well as the upcoming v2.11-rc1.
-Peff
