Re: [PATCH net-next v7 12/12] selftests: ncdevmem: Add automated test

From: Jakub Kicinski
Date: Mon Nov 04 2024 - 20:34:11 EST


On Mon, 4 Nov 2024 16:15:10 -0800 Joe Damato wrote:
> > + with bkg(listen_cmd) as nc:
> > + wait_port_listen(port)
> > + cmd(f"echo -e \"hello\\nworld\"| nc {cfg.v6} {port}", host=cfg.remote, shell=True)
>
> FWIW, in the v3 of the series I submit, Jakub asked me to replace nc
> with socat due to issues with nc [1].
>
> Your usage of nc seems pretty basic though, so maybe it's fine?

Good catch, let's not use nc. I seem to recall it's also funny about
address family selection. Also may be useful to add a helper for sending
simple strings from remote to avoid having this problem again.