wait_terminate timeout
Feedback from ui/daiquiri!457 (comment 166573)
In https://gitlab.esrf.fr/bliss/bliss/-/blob/master/bliss/testutils/process_utils.py#L56
There is twice a timeout
def wait_terminate(process, timeout=10):
...
try:
with gevent.Timeout(timeout):
...
except gevent.Timeout:
process.kill()
with gevent.Timeout(10):
...
What about:
- Using twice
Timeout(timeout)
instead of hardcoded 10 - Or twice
Timeout(timeout / 2)
, which is i think a bit more consistent with the user request
What do you think @matias.guijarro @denolf @marcus.oscarsson ?