On Mon, Nov 07, 2022 at 02:21:30PM +1100, David Gibson wrote:
> On Fri, Nov 04, 2022 at 02:53:36AM +0100, Stefano Brivio wrote:
> > As pasta now configures that target network namespace with
> > --config-net, we need to wait for addresses and routes to be actually
> > present. Just sending netlink messages doesn't mean this is done
> > synchronously.
> > 
> > A more elegant alternative, which probably makes sense regardless of
> > this test setup, would be to query, from pasta, addresses and routes
> > we added, and wait until they're there, before proceeding.
> > 
> > Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
> 
> Ugly, but I see the point.

Oops, forgot to add

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

> 
> As you say in future I hope we can actually detect the completion of
> interface configuration which would be useful both here and other
> places.
> 
> One other thing to consider - in the case where pasta is spawing a new
> process, rather than attaching to an existing one, it's possibly
> counterintuitive that the process starts before pasta has completed
> setup.  We possibly should get the spawned thread to wait until the
> main pasta thread is entering its main loop before exec()ing the
> shell.  Obviously requires some sync mechanism, which is a bit clunky.
> 
> > ---
> >  test/lib/setup | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/test/lib/setup b/test/lib/setup
> > index ab6d8d5..07d5056 100755
> > --- a/test/lib/setup
> > +++ b/test/lib/setup
> > @@ -208,6 +208,8 @@ setup_two_guests() {
> >  
> >  	__ifname="$(context_run qemu_1 "ip -j link show | jq -rM '.[] | select(.link_type == \"ether\").ifname'")"
> >  
> > +	sleep 1
> > +
> >  	__opts=
> >  	[ ${PCAP} -eq 1 ] && __opts="${__opts} -p ${LOGDIR}/passt_1.pcap"
> >  	[ ${DEBUG} -eq 1 ] && __opts="${__opts} -d"
> 



-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson