storage autopkgtest always fails with linux-{kvm,azure,etc} kernels that don't include scsi_debug module
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
autopkgtest-cloud |
Fix Released
|
Undecided
|
Dan Streetman | ||
linux-kvm (Ubuntu) |
New
|
Undecided
|
Unassigned | ||
Xenial |
New
|
Undecided
|
Unassigned | ||
Bionic |
New
|
Undecided
|
Unassigned | ||
Disco |
Won't Fix
|
Undecided
|
Unassigned | ||
Eoan |
New
|
Undecided
|
Unassigned | ||
Focal |
New
|
Undecided
|
Unassigned | ||
systemd (Ubuntu) |
Fix Released
|
Low
|
Dan Streetman | ||
Xenial |
Fix Released
|
Low
|
Dan Streetman | ||
Bionic |
Fix Released
|
Low
|
Dan Streetman | ||
Disco |
Won't Fix
|
Low
|
Dan Streetman | ||
Eoan |
Fix Released
|
Low
|
Dan Streetman | ||
Focal |
Fix Released
|
Low
|
Dan Streetman |
Bug Description
Note: this bug touches on multiple packages:
- autopkgtest-cloud: merge request open to install 'modules-extra' package
- systemd: update autopkgtest to skip test if scsi_debug module isn't available (SRU template below is for this change)
- linux-kvm: has CONFIG_SCSI_DEBUG unset, but would be good to have scsi_debug module built for this kernel
[SRU template for systemd]
[impact]
the 'storage' autopkgtest requires the 'scsi_debug' module, and if it can't load it (and it's not built-in), the test fails. On some custom kernels, this module is included in the new 'linux-
Additionally, some custom kernels, like linux-kvm, have SCSI_DEBUG disabled entirely; for those kernels, it will never be possible to load the scsi_debug module and the test should be skipped.
[test case]
run the 'storage' autopkgtest with a custom kernel that doesn't provide the scsi_debug module in the standard linux-modules deb, or check the autopkgtest.u.c history, for example:
[regression potential]
as this modifies one of the autopkgtest --setup-commands, a regression would most likely cause immediate test failure due to failing to install the proper linux-* debs.
For the systemd 'storage' test change, this only skips the test if the scsi_debug module can't be loaded (and isn't already present, i.e. loaded or built-in). Any regression potential from that is low, but likely would involve incorrect test skips and/or missing valid problems due to skipping the test.
[scope]
systemd in all releases needs to have its 'storage' test updated
Related branches
- Iain Lane (community): Approve
- Andy Whitcroft: Approve
- Seth Forshee (community): Needs Fixing
-
Diff: 12 lines (+1/-0)1 file modifiedworker/worker (+1/-0)
Changed in systemd (Ubuntu Eoan): | |
importance: | Undecided → Low |
Changed in systemd (Ubuntu Disco): | |
importance: | Undecided → Low |
Changed in systemd (Ubuntu Bionic): | |
importance: | Undecided → Low |
Changed in systemd (Ubuntu Xenial): | |
importance: | Undecided → Low |
Changed in systemd (Ubuntu Eoan): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Disco): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Bionic): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Xenial): | |
status: | New → In Progress |
Changed in systemd (Ubuntu): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Eoan): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Disco): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Bionic): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Xenial): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Disco): | |
status: | In Progress → Won't Fix |
description: | updated |
Changed in autopkgtest-cloud: | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Focal): | |
assignee: | Dan Streetman (ddstreet) → nobody |
Changed in systemd (Ubuntu Eoan): | |
assignee: | Dan Streetman (ddstreet) → nobody |
Changed in systemd (Ubuntu Disco): | |
assignee: | Dan Streetman (ddstreet) → nobody |
Changed in systemd (Ubuntu Xenial): | |
assignee: | Dan Streetman (ddstreet) → nobody |
Changed in systemd (Ubuntu Bionic): | |
assignee: | Dan Streetman (ddstreet) → nobody |
Changed in systemd (Ubuntu Xenial): | |
status: | In Progress → Invalid |
Changed in systemd (Ubuntu Bionic): | |
status: | In Progress → Invalid |
Changed in systemd (Ubuntu Eoan): | |
status: | In Progress → Invalid |
Changed in systemd (Ubuntu Focal): | |
status: | In Progress → Invalid |
Changed in systemd (Ubuntu Disco): | |
status: | Won't Fix → Invalid |
description: | updated |
description: | updated |
Changed in systemd (Ubuntu Focal): | |
status: | Invalid → In Progress |
Changed in systemd (Ubuntu Disco): | |
status: | Invalid → Won't Fix |
Changed in systemd (Ubuntu Eoan): | |
status: | Invalid → In Progress |
Changed in systemd (Ubuntu Bionic): | |
status: | Invalid → In Progress |
Changed in systemd (Ubuntu Xenial): | |
status: | Invalid → In Progress |
Changed in systemd (Ubuntu Focal): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Eoan): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Disco): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Bionic): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Xenial): | |
assignee: | nobody → Dan Streetman (ddstreet) |
description: | updated |
description: | updated |
description: | updated |
Changed in systemd (Ubuntu Bionic): | |
status: | Fix Committed → In Progress |
Changed in systemd (Ubuntu Eoan): | |
status: | Fix Committed → In Progress |
Changed in autopkgtest-cloud: | |
status: | New → Fix Released |
Changed in linux-kvm (Ubuntu Disco): | |
status: | New → Won't Fix |
This bug was fixed in the package systemd - 244.1-0ubuntu2
---------------
systemd (244.1-0ubuntu2) focal; urgency=medium
[ Dimitri John Ledkov ] patches/ shutdown- do-not- detach- autoclear- loopback- devices. patch /git.launchpad. net/~ubuntu- core-dev/ ubuntu/ +source/ systemd/ commit/ ?id=3978d34b59e 98cdd01836c41a1 0442967636b8fc
* shutdown: do not detach autoclear loopback devices
Author: Dimitri John Ledkov
File: debian/
https:/
[ Balint Reczey ] patches/ Revert- network- if-sys- is-rw-then- udev-should- be-around. patch /git.launchpad. net/~ubuntu- core-dev/ ubuntu/ +source/ systemd/ commit/ ?id=39c12f8e736 afd1b7bdeb13ff6 bccaea85020873
* Revert upstream commit breaking IPv4 DHCP in LXC containers in 244.1
(LP: #1857123)
File: debian/
https:/
systemd (244.1-0ubuntu1) focal; urgency=medium
* New upstream version 244.1 patches/ lp-1853852- seccomp- fix-multiplexed -system- calls.patch patches/ lp-1853852- seccomp- mmap-test- results- depend- on-kernel- libseccom. patch /git.launchpad. net/~ubuntu- core-dev/ ubuntu/ +source/ systemd/ commit/ ?id=382271662c6 0c339b0a404c7a1 772fe5670516ef network- set-ipv6- mtu-after- link-up- or-device- mtu-cha. patch: pstore. service in containers. and-services autopkgtest fail. (LP: #1856729) patches/ pstore- Don-t-start- systemd- pstore. service- in-containers. patch /git.launchpad. net/~ubuntu- core-dev/ ubuntu/ +source/ systemd/ commit/ ?id=28b5a03769c bed9d3170ebac38 508b867530a2d6 patches/ Revert- network- do-not- drop-foreign- config- if-interface- is.patch /git.launchpad. net/~ubuntu- core-dev/ ubuntu/ +source/ systemd/ commit/ ?id=22a9fa3bb03 ba2a629926af39e a7df81fe33c9b8
- network: set ipv6 mtu after link-up or device mtu change (LP: #1671951)
- & other changes
* Refresh patches.
- Dropped changes:
* d/p/lp-1853852-*: fix issues with muliplexed shmat calls (LP: #1853852)
Files:
- debian/
- debian/
https:/
* d/p/lp1671951-
set ipv6 mtu at correct time
* pstore: Don't start systemd-
Usually it is not useful and can also fail making
boot-
File: debian/
https:/
* Revert: network: do not drop foreign config if interface is in initialized state.
This fixes FTBFS with the other network-related reverts.
File: debian/
https:/
systemd (244-3ubuntu5) focal; urgency=medium
[ Dariusz Gadomski ] 0001-user- util-Add- helper- functions- for-gid- lists-operati. patch, lp1762391/ 0002-execute- Restore- call-to- pam_setcred. patch, lp1762391/ 0003-execute- Detect- groups- added-by- PAM-and- merge-them- wi.patch, lp1762391/ 0004-test- Add-tests- for-gid- list-ops. patch, lp1762391/ 0005-execute- add-const- to-array- parameters- where-possible. patch, lp1762391/ 0006-execute- allow-pam_ setcred- to-fail- ignore- errors. patch:
* d/p/lp1762391/
d/p/
d/p/
d/p/
d/p/
d/p/
- Restore call to pam_setcred (LP: #1762391)
[ Dan Streetman ]
* d/t/storage: without scsi_debug, skip test (LP: #1847816)
systemd (244-3ubuntu4) focal; urgency=medium
* d/p/lp1671951- network- set-ipv6- mtu-after- link-up- or-device- mtu-cha. patch: network- rename- linux_configure _after_ setting_ mtu-to- linux.patch, 0002-network- add-link- setting_ genmode- flag.patc. ..
set ipv6 mtu at correct time (LP: #1671951)
* d/p/0001-
d/p/