Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(189)

Unified Diff: tools/telemetry/telemetry/testing/run_tests.py

Issue 1379063003: [Telemetry] Move listing stray processes to more appropriate places (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/telemetry/telemetry/internal/util/ps_util.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/telemetry/telemetry/testing/run_tests.py
diff --git a/tools/telemetry/telemetry/testing/run_tests.py b/tools/telemetry/telemetry/testing/run_tests.py
index e40fda2a9ce9889d0d945a99346632402e6a6932..d05602743cb10e13c964084c1814f1aa9e7d07f9 100644
--- a/tools/telemetry/telemetry/testing/run_tests.py
+++ b/tools/telemetry/telemetry/testing/run_tests.py
@@ -13,6 +13,7 @@ from telemetry.internal.browser import browser_options
from telemetry.internal.platform import device_finder
from telemetry.internal.util import binary_manager
from telemetry.internal.util import command_line
+from telemetry.internal.util import ps_util
from telemetry.testing import browser_test_case
from telemetry.testing import options_for_unittests
@@ -206,12 +207,18 @@ def _MatchesSelectedTest(name, selected_tests, selected_tests_are_exact):
def _SetUpProcess(child, context): # pylint: disable=W0613
+ ps_util.EnableListingStrayProcessesUponExitHook()
if binary_manager.NeedsInit():
# Typ doesn't keep the DependencyManager initialization in the child
# processes.
binary_manager.InitDependencyManager(context.client_config)
# We need to reset the handlers in case some other parts of telemetry already
# set it to make this work.
+ logging.getLogger().handlers = []
+ logging.basicConfig(
+ level=logging.INFO,
+ format='(%(levelname)s) %(asctime)s %(module)s.%(funcName)s:%(lineno)d '
+ '%(message)s')
args = context
if not args.disable_logging_config:
logging.getLogger().handlers = []
« no previous file with comments | « tools/telemetry/telemetry/internal/util/ps_util.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698