On Tue, Feb 10, 2026 at 10:25 AM Dumitru Ceara <[email protected]> wrote:
> On 2/10/26 9:03 AM, Ales Musil wrote: > > The multinode was using $PYTHON, but that's not a valid path > > inside the container. However, the script is executable and has > > the proper shebang so we can run it directly. > > > > Signed-off-by: Ales Musil <[email protected]> > > --- > > Hi Ales, > > > tests/multinode-macros.at | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/tests/multinode-macros.at b/tests/multinode-macros.at > > index 31dc00fe6..c4415ce1c 100644 > > --- a/tests/multinode-macros.at > > +++ b/tests/multinode-macros.at > > @@ -476,9 +476,9 @@ m_is_fedora() { > > m4_define([M_START_L4_SERVER], > > [podman cp $srcdir/server.py $1:/data/metadata_server.py || exit 1 > > M_NS_DAEMONIZE([$1], [$2], > > - [$PYTHON /data/metadata_server.py --bind-host $3 \ > > - --bind-port $4 \ > > - --reply-string > $5], > > + [./data/metadata_server.py --bind-host $3 \ > > + --bind-port $4 \ > > + --reply-string $5], > > [$6]) > > pid=$(podman exec $1 ip netns exec $2 ps aux | grep > metadata_server.py | grep $5 | tr -s ' ' | cut -d' ' -f2) > > on_exit "podman exec $1 ip netns exec $2 kill $pid" > > Looks good to me, thanks for the fix! We could add a "Fixes" tag I > guess but regardless: > > Acked-by: Dumitru Ceara <[email protected]> > > Regards, > Dumitru > > Thank you Dumitru, I went ahead and merged this into main with the fixes tag added. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
