Behaviour change in systemd 243 breaks libvirt autopkgtest
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
systemd |
New
|
Unknown
|
|||
libvirt (Ubuntu) |
Invalid
|
Medium
|
Christian Ehrhardt | ||
systemd (Ubuntu) |
Fix Released
|
High
|
Unassigned |
Bug Description
With systemd 243 libvirt autopkgtest starts failing:
http://
https:/
...
+ virt-xml-validate debian/
debian/
+ virsh define debian/
Domain sl defined from debian/
+ rm -f /var/log/
+ virsh start sl
Domain sl started
+ grep -qs starting up /var/log/
+ check_domain
+ grep/bin/ls
Domain sl has been undefined
-qs sl[[:space:
+ virsh list
+ virsh lxc-enter-namespace --noseclabel sl /bin/ls /bin/ls
+ systemctl restart libvirtd
+ check_domain
+ grep -qs sl[[:space:
+ virsh list
+ cleanup
+ [ -z ]
+ virsh destroy sl
error: Failed to destroy domain sl
error: Requested operation is not valid: domain is not running
+ true
+ virsh undefine sl
+ CLEANED_UP=1
autopkgtest [10:41:05]: test smoke-lxc: -------
autopkgtest [10:41:07]: test smoke-lxc: - - - - - - - - - - results - - - - - - - - - -
smoke-lxc FAIL non-zero exit status 1
...
The behaviour change is still present in systemd master (v243-112-
I can revert the change in systemd, but it also fixes an issue reported at upstream.
I will take another look later, but wanted to document my findings.
Changed in systemd: | |
status: | Unknown → New |
The commit where the regression is introduced in this commit:
commit 0219b3524f414e2 3589e63c6de6a75 9811ef8474
Author: Donald Buczek <email address hidden>
Date: Thu Apr 25 09:39:41 2019 +0200
cgroup: Continue unit reset if cgroup is busy
When part of the cgroup hierarchy cannot be deleted (e.g. because there
are still processes in it), do not exit unit_prune_cgroup early, but
continue so that u->cgroup_realized is reset.
Log the known case of non-empty cgroups at debug level and other errors
at warning level.
Fixes https:/ /github. com/systemd/ systemd/ issues/ 12386