cyrusimap / cassandane

Other
6 stars 11 forks source link

testrunner: try harder to clean up on ^C #6

Closed elliefm closed 7 years ago

elliefm commented 7 years ago

When a parallel (-j n) testrunner.pl is stopped with ^C, the Cyrus instances that it started up are left running.

If it doesn't already have a handler for SIGINT, one should be installed that shuts the test instances down cleanly.

If it does already have such a handler, then it needs to be fixed because it doesn't work properly.

As a workaround in the meantime, something like sudo -u cyrus killall master usually does the trick (as verified by subsequent ps -ef | grep cyrus showing no processes other than the grep).

elliefm commented 7 years ago

There's probably more underlying weirdness to ferret out over time (IPC is fiddly in perl) but I'm calling this fixed for now