Increase timeout used when waiting for manager to shutdown cleanly

before resorting to terminate()
This commit is contained in:
Richard Oudkerk 2012-06-15 20:08:29 +01:00
parent 0f52346e76
commit 3049f1243e
2 changed files with 6 additions and 1 deletions

View File

@ -582,7 +582,7 @@ class BaseManager(object):
except Exception:
pass
process.join(timeout=0.2)
process.join(timeout=1.0)
if process.is_alive():
util.info('manager still alive')
if hasattr(process, 'terminate'):

View File

@ -1820,6 +1820,11 @@ class _TestZZZNumberOfObjects(BaseTestCase):
# run after all the other tests for the manager. It tests that
# there have been no "reference leaks" for the manager's shared
# objects. Note the comment in _TestPool.test_terminate().
# If some other test using ManagerMixin.manager fails, then the
# raised exception may keep alive a frame which holds a reference
# to a managed object. This will cause test_number_of_objects to
# also fail.
ALLOWED_TYPES = ('manager',)
def test_number_of_objects(self):