diff options
author | Neil Williams <neil.williams@linaro.org> | 2017-11-01 13:54:41 +0000 |
---|---|---|
committer | Neil Williams <neil.williams@linaro.org> | 2017-11-01 14:06:20 +0000 |
commit | d9553680ad631f0236f882f1df72b6187b81a679 (patch) | |
tree | 2845ab4fe7923d12b12f1ccfccfa9da2306c28bb /lava_dispatcher/actions/test | |
parent | 61cff41c6408375cfdd5fc45e22310950ea0f3d0 (diff) |
Add feedback check to test shell
Complete the change for 61cff41c.
Change-Id: If92cae47c4f29eb6228aa990657b2bab4a0ab702
Diffstat (limited to 'lava_dispatcher/actions/test')
-rw-r--r-- | lava_dispatcher/actions/test/shell.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lava_dispatcher/actions/test/shell.py b/lava_dispatcher/actions/test/shell.py index 170272f1e..865590c27 100644 --- a/lava_dispatcher/actions/test/shell.py +++ b/lava_dispatcher/actions/test/shell.py @@ -286,14 +286,14 @@ class TestShellAction(TestAction): # Only grab the feedbacks every test_connection.timeout if feedbacks and time.time() - last_check > test_connection.timeout: for feedback in feedbacks: - self.logger.debug("Listening to namespace '%s'", feedback[0]) # The timeout is really small because the goal is only # to clean the buffer of the feedback connections: # the characters are already in the buffer. # With an higher timeout, this can have a big impact on # the performances of the overall loop. - feedback[1].listen_feedback(timeout=1) - self.logger.debug("Listening to namespace '%s' done", feedback[0]) + bytes_read = feedback[1].listen_feedback(timeout=1) + if bytes_read > 1: + self.logger.debug("Listened to connection for namespace '%s' done", feedback[0]) last_check = time.time() finally: if self.current_run is not None: |