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

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

[patrick]

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

[meta-virtualization][PATCH 2/2] netavark: skip plugins in ptest

[patrick]

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

Re: [meta-virtualization][PATCH 2/2] netavark: skip plugins in ptest

[Yu, Mingli]

The ptest fails as below:

# ./run-ptest
/tmp/bats-run-ELGGaA/bats.18820.src: line 5: To: command not found
1..1
not ok 1 bats-gather-tests
# (in test file test/README.md, line 5)
#   `To run the tests locally in your sandbox, you can use one of these 
methods:' failed with status 127

Thanks,

On 8/25/23 22:51, patrick@subset.ch wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
> 
> 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/3618448
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [mingli.yu@eng.windriver.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 

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

[Bruce Ashfield]

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]
> -=-=-=-=-=-=-=-=-=-=-=-
>