Re: [meta-virtualization][PATCH 1/2] netavark: require iputils for ptest

[Bruce Ashfield <bruce.ashfield@gmail.com>]

Thanks for doing this against master-next!

I'm out of the office for another week, but have put
this onto master-next and will complete the merges
once I'm fully back to the office.

Bruce

In message: [meta-virtualization][PATCH 1/2] netavark: require iputils for ptest
on 25/08/2023 patrick@subset.ch wrote:

> From: Patrick Wicki <patrick.wicki@siemens.com>
> 
> Fixes ptest not working on systems using busybox ping.
> 
> Signed-off-by: Patrick Wicki <patrick.wicki@siemens.com>
> ---
>  recipes-containers/netavark/netavark_1.7.0.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/recipes-containers/netavark/netavark_1.7.0.bb b/recipes-containers/netavark/netavark_1.7.0.bb
> index 43c3ee5f..490d8762 100644
> --- a/recipes-containers/netavark/netavark_1.7.0.bb
> +++ b/recipes-containers/netavark/netavark_1.7.0.bb
> @@ -44,6 +44,7 @@ DEPENDS += "protobuf-c-native protobuf-c"
>  # bind-utils is used to install dig
>  # procps-ps is necessary because the ps from busybox is
>  # not having the same behavior
> +# iputils is used because busybox ping behaves differently
>  RDEPENDS:${PN}-ptest += " \
>      bash \
>      bats \
> @@ -51,6 +52,7 @@ RDEPENDS:${PN}-ptest += " \
>      coreutils \
>      dbus-daemon-proxy \
>      iproute2 \
> +    iputils \
>      jq \
>      nmap \
>      procps-ps \
> -- 
> 2.41.0
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#8231): https://lists.yoctoproject.org/g/meta-virtualization/message/8231
> Mute This Topic: https://lists.yoctoproject.org/mt/100956964/1050810
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


In message: [meta-virtualization][PATCH 2/2] netavark: skip plugins in ptest
on 25/08/2023 patrick@subset.ch wrote:

> From: Patrick Wicki <patrick.wicki@siemens.com>
> 
> Skip the plugin tests that were introduced in 1.6.0 and lead to ptest
> failure.
> 
> Signed-off-by: Patrick Wicki <patrick.wicki@siemens.com>
> ---
>  recipes-containers/netavark/files/run-ptest | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/recipes-containers/netavark/files/run-ptest b/recipes-containers/netavark/files/run-ptest
> index 41a37dce..7e017ae2 100644
> --- a/recipes-containers/netavark/files/run-ptest
> +++ b/recipes-containers/netavark/files/run-ptest
> @@ -1,3 +1,8 @@
>  #!/bin/bash
>  
> -NETAVARK=/usr/libexec/podman/netavark bats ./test/
> +shopt -s extglob
> +
> +# Skip the plugin tests because those example plugins are built only while
> +# running cargo test and for this to work as a ptest they would need to be cross
> +# compiled and installed as part of the ptest.
> +NETAVARK=/usr/libexec/podman/netavark bats test/!(*-plugin.bats)
> -- 
> 2.41.0
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#8232): https://lists.yoctoproject.org/g/meta-virtualization/message/8232
> Mute This Topic: https://lists.yoctoproject.org/mt/100956965/1050810
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>