[ARVADOS] updated: 1.3.0-11-g734de92e6

Git user git at public.curoverse.com
Mon Dec 10 11:21:04 EST 2018


Summary of changes:
 sdk/cwl/arvados_cwl/executor.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  734de92e6f0254d21a1b9d29c634804c44e77ca6 (commit)
      from  cbc55ee5ac64597cb0856497d3dedeccec79f440 (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 734de92e6f0254d21a1b9d29c634804c44e77ca6
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Mon Dec 10 11:20:41 2018 -0500

    14476: Remove redundant falsey check
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/sdk/cwl/arvados_cwl/executor.py b/sdk/cwl/arvados_cwl/executor.py
index 195eda695..27774b2f7 100644
--- a/sdk/cwl/arvados_cwl/executor.py
+++ b/sdk/cwl/arvados_cwl/executor.py
@@ -726,7 +726,7 @@ http://doc.arvados.org/install/install-api-server.html#disable_api_methods
                                                      body={"state": "Failed"}).execute(num_retries=self.num_retries)
             if runtimeContext.submit and isinstance(tool, Runner):
                 runnerjob = tool
-                if runnerjob and runnerjob.uuid and self.work_api == "containers":
+                if runnerjob.uuid and self.work_api == "containers":
                     self.api.container_requests().update(uuid=runnerjob.uuid,
                                                      body={"priority": "0"}).execute(num_retries=self.num_retries)
         finally:

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list