[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