[ARVADOS] updated: 5d8e0b2caf01c6d78f7de20382742b6516697cd9

git at public.curoverse.com git at public.curoverse.com
Wed Aug 20 15:05:00 EDT 2014


Summary of changes:
 sdk/cli/bin/crunch-job | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

       via  5d8e0b2caf01c6d78f7de20382742b6516697cd9 (commit)
      from  05bea2c50474edeb9d0e3fb8daaf838b58ea9a54 (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 5d8e0b2caf01c6d78f7de20382742b6516697cd9
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Aug 20 15:04:33 2014 -0400

    Treat success=true exitcode!=0 as success when collating output,
    just like we did when we decided the task completed. No issue #

diff --git a/sdk/cli/bin/crunch-job b/sdk/cli/bin/crunch-job
index e7cac18..47f9894 100755
--- a/sdk/cli/bin/crunch-job
+++ b/sdk/cli/bin/crunch-job
@@ -1223,9 +1223,8 @@ sub collate_output
   my $joboutput;
   for (@jobstep)
   {
-    next if (!exists $_->{'arvados_task'}->{output} ||
-             !$_->{'arvados_task'}->{'success'} ||
-             $_->{'exitcode'} != 0);
+    next if (!exists $_->{'arvados_task'}->{'output'} ||
+             !$_->{'arvados_task'}->{'success'});
     my $output = $_->{'arvados_task'}->{output};
     if ($output !~ /^[0-9a-f]{32}(\+\S+)*$/)
     {

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list