There's some code/test I'm going breakrelating to shutdown and single deployment.

It was DeployerSingleDeploymentTestCase::testMultiThreadsAndShutdown()

This is incorrect behaviour.

The shutdown should not be blocked by a deployment, it wereit could "deadlock" the shutdown when the deployment is misbehaving (e.g. a broken servicethat is waiting on a socket forever because of a network split)

I'm going to remove this test.

The test actually suffers from a race anyway.The deployment threads (as written currently) could run after the shutdown threadgets the lock which means the test would fail.