Re: Mock - signal reaction
Well, this is a question from Miroslav Suchý on my PR with a solution. https://github.com/rpm-software-management/mock/pull/293 ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: Mock - signal reaction
On Tuesday, July 30, 2019 4:59:13 PM CEST Jan Buchmaier wrote: > Do we want to catch/process SIGABRT as well? I see no obvious use case for catching SIGABRT by mock. Why are you asking about SIGABRT specifically? Kamil ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: Mock - signal reaction
Do we want to catch/process SIGABRT as well? ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: Mock - signal reaction
On Tuesday, June 4, 2019 9:33:33 AM CEST Miroslav Suchý wrote: > Dne 03. 06. 19 v 13:00 Kamil Dudka napsal(a): > > >> Now I'm working on signal SIGTERM handling and I would like to kill all > >> processes related to the main mock process. > >> > >> What do you think is it a good > >> > >> idea to kill all processes, or do we want to kill the main process only? > >> And what about SIGINT so-called KeyInterrupt in python? Same reaction? > > > > Your question is missing some context. Are you trying to improve signal > > handling in your own code that uses mock, or are you trying to improve > > signal handling in the implementation of mock itself? > > > I will try to clarify this. > > In past, when you send SIGKILL to Mock, then only Mock terminated. It leaves > behind any running rpmbuild or dnf. > Jan is working on fix for this use > case: > https://github.com/rpm-software-management/mock/issues/153 > As part of this work, we asked ourself the question: should mock clean up > after itself when exiting? I think it should. But you will need to handle SIGTERM instead of SIGKILL. > But I wonder what should be correct/expected behaviour? E.g., is someone > running daemons in mock using 'chroot' or > 'shell' command and would be > surprised if Mock will kill it when Mock will terminate? Not that I know of. > Or we can assume > that when Mock terminate then nothing should be running in the chroot? Yes please. > What are your use-cases? csmock uses it for fully automated static analysis of RPM packages. Kamil ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: Mock - signal reaction
Dne 03. 06. 19 v 13:00 Kamil Dudka napsal(a): >> Now I'm working on signal SIGTERM handling and I would like to kill all >> processes related to the main mock process. >> What do you think is it a good >> idea to kill all processes, or do we want to kill the main process only? >> And what about SIGINT so-called KeyInterrupt in python? Same reaction? > Your question is missing some context. Are you trying to improve signal > handling in your own code that uses mock, or are you trying to improve > signal handling in the implementation of mock itself? I will try to clarify this. In past, when you send SIGKILL to Mock, then only Mock terminated. It leaves behind any running rpmbuild or dnf. Jan is working on fix for this use case: https://github.com/rpm-software-management/mock/issues/153 As part of this work, we asked ourself the question: should mock clean up after itself when exiting? But I wonder what should be correct/expected behaviour? E.g., is someone running daemons in mock using 'chroot' or 'shell' command and would be surprised if Mock will kill it when Mock will terminate? Or we can assume that when Mock terminate then nothing should be running in the chroot? What are your use-cases? -- Miroslav Suchy, RHCA Red Hat, Associate Manager ABRT/Copr, #brno, #fedora-buildsys ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: Mock - signal reaction
On Monday, June 3, 2019 12:18:02 PM CEST Jan Buchmaier wrote: > Now I'm working on signal SIGTERM handling and I would like to kill all > processes related to the main mock process. > What do you think is it a good > idea to kill all processes, or do we want to kill the main process only? > And what about SIGINT so-called KeyInterrupt in python? Same reaction? Your question is missing some context. Are you trying to improve signal handling in your own code that uses mock, or are you trying to improve signal handling in the implementation of mock itself? Kamil ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Mock - signal reaction
Now I'm working on signal SIGTERM handling and I would like to kill all processes related to the main mock process. What do you think is it a good idea to kill all processes, or do we want to kill the main process only? And what about SIGINT so-called KeyInterrupt in python? Same reaction? ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org