Skip to content

Commit 98ba8a4

Browse files
authored
use put_nowait for message to stop resource managers (#23)
1 parent 1d2ec45 commit 98ba8a4

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

lymbo/run.py

+7-4
Original file line numberDiff line numberDiff line change
@@ -69,23 +69,26 @@ def run_test_plan(test_plan: TestPlan, max_workers: Optional[int] = None) -> int
6969
run_tests_with_scopes_and_shared_queue, test_plan
7070
)
7171

72-
for r in execresult:
73-
pass # TODO log result
72+
for worker_result in execresult:
73+
logger().debug(f"run_test_plan - worker result: [{worker_result}]")
7474

7575
# should already be 0 but we force this value because this is what stop the resources manager processes
7676
with scopes[LYMBO_TEST_SCOPE_GLOBAL]["lock"]:
7777
scopes[LYMBO_TEST_SCOPE_GLOBAL]["count"] = 0
7878

7979
for _ in range(max_workers if max_workers else 4):
80-
shared_queue.put({"stop": True})
80+
shared_queue.put_nowait({"stop": True})
8181

8282
# Wait for a maximum of 30 seconds for all resource managers to complete
8383
try:
8484
for future in concurrent.futures.as_completed(
8585
resources_manager_futures, timeout=30
8686
):
8787
try:
88-
_ = future.result() # TODO log result
88+
resource_manager_result = future.result()
89+
logger().debug(
90+
f"run_test_plan - resource manager result: [{resource_manager_result}]"
91+
)
8992
except concurrent.futures.TimeoutError:
9093
logger().debug(
9194
"run_test_plan - A resource manager process timed out while waiting for completion."

pyproject.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ classifiers = [
1515
"Development Status :: 4 - Beta",
1616
"Intended Audience :: Developers",
1717
"License :: OSI Approved :: Apache Software License",
18-
"Operating System :: POSIX :: Linux",
18+
"Operating System :: OS Independent",
1919
"Programming Language :: Python :: 3",
2020
"Programming Language :: Python :: 3.9",
2121
"Programming Language :: Python :: 3.10",

0 commit comments

Comments
 (0)