[ARVADOS] updated: 81c3241d08ced11ea118c7f68db62032ad5bc469

git at public.curoverse.com git at public.curoverse.com
Mon Oct 20 16:40:46 EDT 2014


Summary of changes:
 sdk/python/arvados/commands/copy.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

       via  81c3241d08ced11ea118c7f68db62032ad5bc469 (commit)
      from  8bd432b7fe67766d6f92902e20b5e63c9f18146d (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 81c3241d08ced11ea118c7f68db62032ad5bc469
Author: Tim Pierce <twp at curoverse.com>
Date:   Mon Oct 20 16:24:07 2014 -0400

    3699: bug fixes
    
    * abort() should not crash the program because it's calling logging.info
      wrong
    * if the pipeline description is None, correctly substitute an empty
      string in the destination pipeline.
    * finish writing collection progress at 100%

diff --git a/sdk/python/arvados/commands/copy.py b/sdk/python/arvados/commands/copy.py
index 191139e..c9e28b5 100755
--- a/sdk/python/arvados/commands/copy.py
+++ b/sdk/python/arvados/commands/copy.py
@@ -214,7 +214,8 @@ def copy_pipeline_instance(pi_uuid, src, dst, args):
     # instance at dst.
     pi['properties']['copied_from_pipeline_instance_uuid'] = pi_uuid
     pi['description'] = "Pipeline copied from {}\n\n{}".format(
-        pi_uuid, pi.get('description', ''))
+        pi_uuid,
+        pi['description'] if pi.get('description', None) else '')
     if args.project_uuid:
         pi['owner_uuid'] = args.project_uuid
     else:
@@ -248,7 +249,8 @@ def copy_pipeline_template(pt_uuid, src, dst, args):
         copy_git_repos(pt, src, dst, args.dst_git_repo)
 
     pt['description'] = "Pipeline template copied from {}\n\n{}".format(
-        pt_uuid, pt.get('description', ''))
+        pt_uuid,
+        pt['description'] if pt.get('description', None) else '')
     pt['name'] = "{} copied from {}".format(pt.get('name', ''), pt_uuid)
     del pt['uuid']
     del pt['owner_uuid']
@@ -447,6 +449,7 @@ def copy_collection(obj_uuid, src, dst, args):
             dst_manifest += "\n"
 
     if progress_writer:
+        progress_writer.report(obj_uuid, bytes_written, bytes_expected)
         progress_writer.finish()
 
     # Copy the manifest and save the collection.
@@ -556,7 +559,7 @@ def uuid_type(api, object_uuid):
     return None
 
 def abort(msg, code=1):
-    logger.info("arv-copy:", msg)
+    logger.info("arv-copy: %s", msg)
     exit(code)
 
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list