1 From 0d0ca00e56024a9919c150dbed62050d4c70b0c8 Mon Sep 17 00:00:00 2001
2 From: =?UTF-8?q?C=C3=A9dric=20Ollivier?= <cedric.ollivier@orange.com>
3 Date: Wed, 3 Jun 2020 15:23:59 +0200
4 Subject: [PATCH] Switch to threading.Thread() for Rally tasks
6 Content-Type: text/plain; charset=UTF-8
7 Content-Transfer-Encoding: 8bit
9 multiprocessing.Process() often fails due to thread crashes [1].
10 It looks similar to gsutil release notes [2].
12 [1] https://build.opnfv.org/ci/job/functest-opnfv-functest-benchmarking-cntt-latest-rally_full_cntt-run/35/console
13 [2] https://github.com/GoogleCloudPlatform/gsutil/issues/548
14 [3] https://github.com/GoogleCloudPlatform/gsutil/blob/master/CHANGES.md
16 Change-Id: I582933832e23d188c7fa5999e713dd5d7e82d2da
17 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
19 rally/task/runner.py | 7 ++++---
20 1 file changed, 4 insertions(+), 3 deletions(-)
22 diff --git a/rally/task/runner.py b/rally/task/runner.py
23 index 3397e1193..5edebb406 100644
24 --- a/rally/task/runner.py
25 +++ b/rally/task/runner.py
26 @@ -17,6 +17,7 @@ import abc
29 import multiprocessing
33 from rally.common import logging
34 @@ -186,9 +187,9 @@ class ScenarioRunner(plugin.Plugin, validation.ValidatablePluginMixin,
35 for i in range(processes_to_start):
36 kwrgs = {"processes_to_start": processes_to_start,
37 "processes_counter": i}
38 - process = multiprocessing.Process(target=worker_process,
39 - args=next(worker_args_gen),
40 - kwargs={"info": kwrgs})
41 + process = threading.Thread(target=worker_process,
42 + args=next(worker_args_gen),
43 + kwargs={"info": kwrgs})
45 process_pool.append(process)