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
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
