[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Failed with output: Hydra job gnu:coreutils-master:build on x86_64-l

From: Bernhard Voelker
Subject: Re: Failed with output: Hydra job gnu:coreutils-master:build on x86_64-linux
Date: Mon, 03 Jun 2013 13:14:05 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5

Hi Padraig,

thanks again for working on this.

On 06/03/2013 12:13 PM, Pádraig Brady wrote:
> From 12b6c3bf70dde23c2c1bf5db7650a959f327b04f Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?P=C3=A1draig=20Brady?= <address@hidden>
> Date: Mon, 3 Jun 2013 01:29:17 +0100
> Subject: [PATCH] tests: avoid a race in tail --retry testing
> Prompted by the continuous integration build failure at:

Can we rely on this URL to be available quite some time, or is
there some cleanup on Hydra from time to time?

> * tests/tail-2/ Ensure the 'out' file is not present
> as it's used to arbitrate the run order of commands.
> Relying on the truncation in the background tail command,


> is racy because the truncation can occur after the fork
> of the background shell and thus the 'missing' file could
> be created by the time tail(1) looks for it.

I'm not sure I understand the above.
Did you mean

> ---
>  tests/tail-2/ |   11 ++++++-----
>  1 files changed, 6 insertions(+), 5 deletions(-)
> diff --git a/tests/tail-2/ b/tests/tail-2/
> index d56d4c1..070e4fd 100644
> --- a/tests/tail-2/
> +++ b/tests/tail-2/
> @@ -44,9 +44,10 @@ grep -F 'tail: warning: --retry ignored' out || fail=1
>  # === Test:
>  # Ensure that "tail --retry --follow=name" waits for the file to appear.
> +rm out                                       || framework_failure_
>  timeout 10 tail -s.1 --follow=name --retry missing >out 2>&1 & pid=$!

It doesn't look obvious that rm(1) avoids a race, and someone could
easily remove that 'redundant' line again in a future change.
Would you mind to add a short comment there?

Have a nice day,

reply via email to

[Prev in Thread] Current Thread [Next in Thread]