'storage' test needs to wait for systemd-cryptsetup to be active before stopping it
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| systemd (Ubuntu) |
Undecided
|
Unassigned | ||
| Xenial |
Medium
|
Dan Streetman | ||
| Bionic |
Medium
|
Dan Streetman | ||
| Cosmic |
Medium
|
Dan Streetman | ||
| Disco |
Medium
|
Dan Streetman | ||
| Eoan |
Undecided
|
Unassigned |
Bug Description
[impact]
test case fails because it does not wait for the service to become active before stopping it, resulting in failure to rmmod the scsi_debug module because it's still in use.
[test case]
check 'storage' test results for systemd in autopkgtest logs, e.g.:
https:/
[regression potential]
low; test case fix only.
[other info]
detected and reported by @cascardo in bug 1814373 comment 4 and 5, but separate (non-test) systemd bugfix uploaded for that bug so opening this bug to track fixing the test case.
larger fixes/improvements to 'storage' testcase in bug 1829347, but those rejected.
Changed in systemd (Ubuntu Eoan): | |
status: | New → Fix Released |
Changed in systemd (Ubuntu Disco): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Cosmic): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Bionic): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in systemd (Ubuntu Disco): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Cosmic): | |
status: | New → In Progress |
Changed in systemd (Ubuntu Bionic): | |
status: | New → In Progress |
importance: | Undecided → Medium |
Changed in systemd (Ubuntu Cosmic): | |
importance: | Undecided → Medium |
Changed in systemd (Ubuntu Disco): | |
importance: | Undecided → Medium |
tags: | added: ddstreet-next |
Changed in systemd (Ubuntu Xenial): | |
assignee: | nobody → Dan Streetman (ddstreet) |
importance: | Undecided → Medium |
status: | New → In Progress |
tags: | added: systemd |
tags: | removed: ddstreet-next |
Dan Streetman (ddstreet) wrote : | #1 |
Changed in systemd (Ubuntu Disco): | |
status: | In Progress → Won't Fix |
Changed in systemd (Ubuntu Cosmic): | |
status: | In Progress → Won't Fix |
Hello Dan, or anyone else affected,
Accepted systemd into bionic-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.
Changed in systemd (Ubuntu Bionic): | |
status: | In Progress → Fix Committed |
tags: | added: verification-needed verification-needed-bionic |
All autopkgtests for the newly accepted systemd (237-3ubuntu10.34) for bionic have finished running.
The following regressions have been reported in tests triggered by the package:
php7.2/
openssh/
dovecot/
gvfs/1.
Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUp
https:/
[1] https:/
Thank you!
Changed in systemd (Ubuntu Bionic): | |
status: | Fix Committed → In Progress |
Hello Dan, or anyone else affected,
Accepted systemd into bionic-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.
Changed in systemd (Ubuntu Bionic): | |
status: | In Progress → Fix Committed |
All autopkgtests for the newly accepted systemd (237-3ubuntu10.39) for bionic have finished running.
The following regressions have been reported in tests triggered by the package:
php7.2/
gvfs/1.
lxc/3.0.
systemd/
netplan.
Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUp
https:/
[1] https:/
Thank you!
Dan Streetman (ddstreet) wrote : | #6 |
storage test passes.
tags: |
added: verification-done verification-done-bionic removed: verification-needed verification-needed-bionic |
The verification of the Stable Release Update for systemd has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.
Launchpad Janitor (janitor) wrote : | #8 |
This bug was fixed in the package systemd - 237-3ubuntu10.39
---------------
systemd (237-3ubuntu10.39) bionic; urgency=medium
[ Dariusz Gadomski ]
* d/p/lp1762391/
d/p/
d/p/
d/p/
d/p/
d/p/
d/p/
d/p/
- Restore call to pam_setcred (LP: #1762391)
[ Ioanna Alifieraki ]
* d/p/lp1860548/
d/p/
- use snprintf instead of xsprintf (LP: #1860548)
[ Dan Streetman ]
* d/p/lp1833193-
- Update lft when static addr was cfg by dhcp (LP: #1833193)
* d/p/lp1849261/
d/p/
- Only trigger OnFailure= if Restart= is not in effect (LP: #1849261)
* d/p/lp1671951-
- set ipv6 mtu at correct time (LP: #1671951)
* d/p/lp1845909/
d/p/
d/p/
d/p/
d/p/
d/p/
d/p/
d/p/
- if LinkLocalAddres
* d/p/lp1859862-
- enable ipv6 when needed (LP: #1859862)
* d/p/lp1836695-
- (re)add static routes after getting dhcp4 addr (LP: #1836695)
* d/t/storage:
- fix buggy test (LP: #1831459)
- without scsi_debug, skip test (LP: #1847816)
-- Dan Streetman <email address hidden> Thu, 06 Feb 2020 10:00:49 -0500
Changed in systemd (Ubuntu Bionic): | |
status: | Fix Committed → Fix Released |
Hello Dan, or anyone else affected,
Accepted systemd into xenial-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.
Changed in systemd (Ubuntu Xenial): | |
status: | In Progress → Fix Committed |
tags: |
added: verification-needed verification-needed-xenial removed: verification-done |
Ubuntu SRU Bot (ubuntu-sru-bot) wrote : Autopkgtest regression report (systemd/229-4ubuntu21.28) | #10 |
All autopkgtests for the newly accepted systemd (229-4ubuntu21.28) for xenial have finished running.
The following regressions have been reported in tests triggered by the package:
udisks2/
multipath-
nplan/0.32~16.04.7 (amd64)
Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUp
https:/
[1] https:/
Thank you!
Dan Streetman (ddstreet) wrote : | #11 |
autopkgtest passes.
tags: |
added: verification-done verification-done-xenial removed: verification-needed verification-needed-xenial |
Launchpad Janitor (janitor) wrote : | #12 |
This bug was fixed in the package systemd - 229-4ubuntu21.28
---------------
systemd (229-4ubuntu21.28) xenial; urgency=medium
* d/t/logind: skip if nonexistent /sys/power/state (LP: #1862657)
* d/p/lp1839290-
- when restarting service after failure, replace existing queued jobs
(LP: #1839290)
* d/t/storage:
- fix buggy test (LP: #1831459)
- without scsi_debug, skip test (LP: #1847816)
* d/p/lp1867421-
- fix resolution of IntelliMouse (LP: #1867421)
* d/p/lp1858412-
- allow vacuuming journal 'root' dir (LP: #1858412)
* d/p/lp1862232-
- do not let invalid hostname break dhcpv4 (LP: #1862232)
* d/t/systemd-fsckd: Skip test on arm64 (LP: #1870194)
* d/p/lp1837914-
- do not crash if NULL passted to journal destructor (LP: #1837914)
* d/e/initramfs-
- Follow symlinks when finding link files to copy into initramfs
(LP: #1868892)
-- Dan Streetman <email address hidden> Mon, 20 Apr 2020 10:17:40 -0400
Changed in systemd (Ubuntu Xenial): | |
status: | Fix Committed → Fix Released |
as disco is EOL next week, marking this won't fix for disco.