[ARVADOS] updated: 1.1.4-176-gbf50a84
Git user
git at public.curoverse.com
Fri Apr 27 11:55:41 EDT 2018
Summary of changes:
services/nodemanager/tests/integration_test.py | 14 +++++++++++++-
1 file changed, 13 insertions(+), 1 deletion(-)
via bf50a84e584554a0bef6443bc8c36212f3236dcd (commit)
from a221be676f0650993e76a9e402b93fa4ef7b5cf6 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit bf50a84e584554a0bef6443bc8c36212f3236dcd
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date: Fri Apr 27 11:51:51 2018 -0400
Node manager integration test writes to stderr in chunks.
Handles EAGAIN when getting pipe buffer backpressure. no issue #
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>
diff --git a/services/nodemanager/tests/integration_test.py b/services/nodemanager/tests/integration_test.py
index 508e626..1699b57 100755
--- a/services/nodemanager/tests/integration_test.py
+++ b/services/nodemanager/tests/integration_test.py
@@ -21,6 +21,7 @@ import logging
import stat
import tempfile
import shutil
+import errno
from functools import partial
import arvados
import StringIO
@@ -256,7 +257,18 @@ def run_test(name, actions, checks, driver_class, jobs, provider):
logger.info("%s passed", name)
else:
if isinstance(detail_content, StringIO.StringIO):
- sys.stderr.write(detail_content.getvalue())
+ detail_content.seek(0)
+ chunk = detail_content.read(4096)
+ while chunk:
+ try:
+ sys.stderr.write(chunk)
+ chunk = detail_content.read(4096)
+ except IOError as e:
+ if e.errno == errno.EAGAIN:
+ # try again (probably pipe buffer full)
+ pass
+ else:
+ raise
logger.info("%s failed", name)
return code
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list