Re: [PATCH] release/checks/integration: add nc timeout

[Demi Marie Obenour <demiobenour@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1590 bytes --]

On 10/31/25 07:13, Alyssa Ross wrote:
> I recently saw a transient timeout of this test.  Looking at the logs,
> it seems like, before netvm was rebooted, nc was already running, and
> that nc never failed, getting stuck.  Presumably, if it had failed,
> another nc run would have connected successfully.  Avoid this by
> setting a 2 second timeout for nc to ensure this eternal hang can't
> happen.
> 
> Reported-by: Demi Marie Obenour <demiobenour@gmail.com>
> Link: https://spectrum-os.org/lists/archives/spectrum-devel/f52c1aa7-96c0-462b-9287-0dbbae037e4f@gmail.com
> Fixes: ae45978 ("release/checks/networking: init")
> Signed-off-by: Alyssa Ross <hi@alyssa.is>
> ---
>  release/checks/integration/default.nix | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/release/checks/integration/default.nix b/release/checks/integration/default.nix
> index 340fb6e..81db264 100644
> --- a/release/checks/integration/default.nix
> +++ b/release/checks/integration/default.nix
> @@ -19,7 +19,7 @@ let
>      type = "nix";
>      run = writeShellScript "run" ''
>        set -x
> -      while :; do echo hello | ${libressl.nc}/bin/nc -N 10.0.2.2 1234; done
> +      while :; do echo hello | ${libressl.nc}/bin/nc -Nw 2 10.0.2.2 1234; done
>      '';
>    };
>  
> 
> base-commit: b1bb4e5050b09a8edcafc0cf1b1b9c9ab42832bc

That looks good to me.  I would have expected a TCP timeout, but I'm
not surprised it didn't happen.  nc might even explicitly disable
those unless a timeout is set.
-- 
Sincerely,
Demi Marie Obenour (she/her/hers)

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 7253 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]