Dont wait forever after subproces is terminated

eWaterCycle · Oct 26, 2023 · 3fbb905 · 3fbb905
1 parent 7c29974
commit 3fbb905
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -52,6 +52,7 @@ jobs:
     - name: Test with pytest
       run: |
         pytest -vv --cov=grpc4bmi --cov-report xml
+      timeout-minutes: 20
     - name: Correct coverage paths
       run: sed -i "s+$PWD/++g" coverage.xml
     - name: SonarCloud analysis

diff --git a/grpc4bmi/bmi_client_subproc.py b/grpc4bmi/bmi_client_subproc.py
@@ -27,7 +27,7 @@ def __init__(self, module_name, path=None, timeout=None, delay=1):
 
     def __del__(self):
         self.pipe.terminate()
-        self.pipe.wait()
+        self.pipe.wait(timeout=0.1)
 
     def get_value_ref(self, var_name):
         raise NotImplementedError("Cannot exchange memory references across process boundary")