Description of problem: mock --orphanskill does nothing when mock --new-chroot used Version-Release number of selected component: 1.2.3-1 How reproducible: always Steps to Reproduce: 1. First terminal: mock --no-clean --shell --new-chroot "/usr/bin/sleep 100" 2. Second terminal: mock --orphanskill Actual results: Nothing happens Expected results: Killing of running processes in chroot
Hmm with NSPAWN we should call machinectl terminate UUID which should kill all processes in that container.
Yes, I tried to implement this in such way, but there was systemd-nspawn bug (systemd-208). When you terminate machine, next time you run systemd-nspawn with the same UUID it will result in error 251.
We know use generated uuid, so it pretty low chance to hit bug 1142105. And we can work independently. So we fix it on our side and then wait till systemd guys fix their side. Mikhail if you have patch with such fix, I would be happy to review and accept it.
Created attachment 980100 [details] correct orphanskill() when systemd-nspawn used Actually, I have patch, but I am not sure whether it's quite good implementation...
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle. Changing version to '22'. More information and reason for this action is here: https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22
Hmm this does not work. When you use reproducer from #0 then 1) will have different UUID from 2).
Yes, sorry for this, that was absolutely wrong. Should we use some prefix (for ex. "mock-" + uuid) in order to find running containers with machinectl?
I enhanced your original patch and I use machinectl list and machinectl show -pRootDirectory UUID to get UUID of machine, which has the same chrootpath. Fixed in commits: * 674d894 (HEAD -> devel, origin/devel) actually verify chroot path of systemd-nspawn VM [RHBZ#1171737] * 3d04132 for orphanskill() get UUID of systemd-nspawn VM for our chroot [RHBZ#1171737] * 38b10c5 Use 'machinectl terminate' inside orphanskill() when systemd-nspawn used [RHBZ#1171737] Thank you for the initial work.
mock-1.2.13-2.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16053
mock-1.2.13-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16054
mock-1.2.13-2.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16056
mock-1.2.13-2.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8114
mock-1.2.13-2.el6 has been submitted as an update to Fedora EPEL 6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8115
mock-1.2.13-2.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update mock'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16054
mock-1.2.13-2.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update mock'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16056
mock-1.2.13-2.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update mock'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16053
mock-1.2.13-2.el6 has been pushed to the Fedora EPEL 6 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update mock'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8115
mock-1.2.13-2.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update mock'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8114
mock-1.2.13-2.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.2.13-2.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.2.13-2.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.2.13-2.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.2.13-2.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.