net:fcnal-test.sh didn't return a non-zero value even with some sub-tests failed
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubuntu-kernel-tests |
Fix Released
|
Undecided
|
Po-Hsu Lin | ||
linux (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Focal |
Fix Released
|
Low
|
Po-Hsu Lin | ||
Jammy |
Fix Released
|
Low
|
Po-Hsu Lin | ||
Kinetic |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
The net/fcnal-test.sh on F/J won't return a non-zero value even with
some sub test cases failed, here is an example on Jammy:
# Tests passed: 857
# Tests failed: 5
ok 1 selftests: net: fcnal-test.sh
Therefore it's marked as PASSED on our report and making it difficult
to spot these failures:
# With VRF
# SYSCTL: net.ipv4.
# TEST: ping local, VRF bind - VRF IP [FAIL]
# TEST: ping local, device bind - ns-A IP [FAIL]
# SYSCTL: net.ipv4.
# SYSCTL: net.ipv4.
# TEST: ping local, VRF bind - VRF IP [FAIL]
# TEST: ping local, device bind - ns-A IP [FAIL]
# With VRF
# SYSCTL: net.ipv4.
# SYSCTL: net.ipv4.
# SYSCTL: net.ipv4.
# TEST: ping out, vrf device+address bind - ns-B IPv6 LLA [FAIL]
[Fix]
* 0f8a3b48f9 selftests: net/fcnal-test.sh: add exit code
This patch can be cherry-picked into Focal and Jammy kernel.
We don't have this test in Bionic, and this patch has already landed
on Kinetic and OEM-5.17
[Test]
Run the patched test, the return value will be 1 whenever the number
of failed test cases is not 0.
[Where problems could occur]
Change limited to testing tools, but we're expected to see new failures
in Jammy. With bug 2006391 fixed later on, we will see failures
reported in F/K as well.
CVE References
Changed in ubuntu-kernel-tests: | |
status: | New → In Progress |
Changed in linux (Ubuntu Focal): | |
status: | New → In Progress |
Changed in linux (Ubuntu Jammy): | |
status: | New → In Progress |
Changed in ubuntu-kernel-tests: | |
assignee: | nobody → Po-Hsu Lin (cypressyew) |
Changed in linux (Ubuntu Focal): | |
assignee: | nobody → Po-Hsu Lin (cypressyew) |
Changed in linux (Ubuntu Jammy): | |
assignee: | nobody → Po-Hsu Lin (cypressyew) |
Changed in linux (Ubuntu): | |
status: | New → Fix Released |
tags: | added: focal jammy ubuntu-kernel-selftests |
Changed in linux (Ubuntu Kinetic): | |
status: | New → Fix Released |
description: | updated |
description: | updated |
Changed in linux (Ubuntu Jammy): | |
importance: | Undecided → Medium |
Changed in linux (Ubuntu Focal): | |
importance: | Undecided → Low |
Changed in linux (Ubuntu Jammy): | |
importance: | Medium → Low |
status: | In Progress → Fix Committed |
Changed in linux (Ubuntu Focal): | |
status: | In Progress → Fix Committed |
Changed in ubuntu-kernel-tests: | |
status: | In Progress → Fix Released |
https:/ /lists. ubuntu. com/archives/ kernel- team/2023- February/ 136977. html