Comment 8 for bug 193253

Revision history for this message
Stuart Bishop (stub) wrote :

Vincent - I flagged the bzr task as Invalid. I don't see any need at the moment to change bzrlib at the moment as I think you are right and the HTTPServer thread is garbage collected during the teardown.

What I am seeing in our test suite now is a leaking thread and a leaking socket. I tracked down this to a thread spawned by SocketServer.ThreadingMixIn's proces_request() method. This thread never dies - if I make it non-daemonic the test suite never exits. This is as far as I have got - I expect that the handler is blocked waiting on input from a socket that our test suite never sends.

jml - I changed the bug topic rather than open a fresh bug :-) The goal of fixing this bug is to land some test suite updates that stop these sort of leaks happening in the future.