]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
tests: disable process monitor before managers
authorIhar Hrachyshka <ihrachys@redhat.com>
Tue, 8 Sep 2015 09:20:10 +0000 (11:20 +0200)
committerIhar Hrachyshka <ihrachys@redhat.com>
Tue, 8 Sep 2015 09:23:14 +0000 (11:23 +0200)
Otherwise the monitor may respawn managers later, leaving them running.

Issue spotted in:
http://logs.openstack.org/02/216902/4/check/gate-neutron-dsvm-functional/a97df90

Change-Id: I0e68b06c87b5770756fdf7b9201e1986cc67e07b
Related-Bug: #1490051

neutron/tests/functional/agent/linux/test_process_monitor.py

index 0802c3a45891b87248f682a9b62bc5fdbf2bb708..ba64d46c934caa9a7f77df9db74f6a5c891bccc1 100644 (file)
@@ -96,6 +96,7 @@ class BaseTestProcessMonitor(base.BaseTestCase):
             exception=RuntimeError('Not all children respawned.'))
 
     def cleanup_spawned_children(self):
+        self._process_monitor.stop()
         for pm in self._child_processes:
             pm.disable()