<div dir="ltr"><div>Acked-by: Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com">luke.nowakowskikrijger@canonical.com</a>><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 2, 2022 at 10:00 AM Kleber Sacilotto de Souza <<a href="mailto:kleber.souza@canonical.com">kleber.souza@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">BugLink: <a href="https://bugs.launchpad.net/bugs/1982930" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1982930</a><br>
<br>
[Impact]<br>
Testcase ubuntu_kernel_selftests.net:ioam6.sh fails in Focal with the 5.15<br>
kernels with the following errors:<br>
<br>
17:19:35 DEBUG| [stdout] make: Entering directory '/tmp/autopkgtest.WSxPqj/build.GQs/src/autotest/cl<br>
ient/tmp/ubuntu_kernel_selftests/src/linux/tools/testing/selftests/net'<br>
17:19:35 DEBUG| [stdout] make --no-builtin-rules ARCH=x86 -C ../../../.. headers_install<br>
17:19:35 DEBUG| [stdout] make[1]: Entering directory '/tmp/autopkgtest.WSxPqj/build.GQs/src/autotest/client/tmp/ubuntu_kernel_selftests/src/linux'<br>
17:19:35 DEBUG| [stdout] INSTALL ./usr/include<br>
17:19:35 DEBUG| [stdout] make[1]: Leaving directory '/tmp/autopkgtest.WSxPqj/build.GQs/src/autotest/client/tmp/ubuntu_kernel_selftests/src/linux'<br>
17:19:35 DEBUG| [stdout] TAP version 13<br>
17:19:35 DEBUG| [stdout] 1..1<br>
17:19:35 DEBUG| [stdout] # selftests: net: ioam6.sh<br>
17:19:35 DEBUG| [stdout] # SKIP: iproute2 too old, missing ioam command<br>
17:19:35 DEBUG| [stdout] not ok 1 selftests: net: ioam6.sh # exit=1<br>
<br>
This is caused by an older version of iproute2 in Focal. The package has been<br>
updated in Jammy via bug 1951390 and the bug report says it should be updated in<br>
Focal as well. Even if the package is eventually updated, the testcase shouldn't<br>
throw a fatal error in this case and really return "SKIP" as the log message<br>
says.<br>
<br>
[Fix]<br>
The proper fix would be to make tools/testing/selftests/net/ioam6.sh return the<br>
Kselftest framework proper skip code instead of failing.<br>
<br>
[Testcase]<br>
Run the selftest script in Focal with a 5.15 kernel.<br>
<br>
[Where problems could occur]<br>
If the skip code is added to places where it shouldn't be added it could<br>
introduce false positives and not fail where it should.<br>
<br>
<br>
Kleber Sacilotto de Souza (1):<br>
  selftests: net: fix IOAM test skip return code<br>
<br>
 tools/testing/selftests/net/ioam6.sh | 12 +++++++-----<br>
 1 file changed, 7 insertions(+), 5 deletions(-)<br>
<br>
-- <br>
2.34.1<br>
<br>
<br>
-- <br>
kernel-team mailing list<br>
<a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
<a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div></div>