[ARVADOS] updated: 6d84b6c8ebeaf59a40cd9e4db1a7e897bf76511c

git at public.curoverse.com git at public.curoverse.com
Mon Sep 15 13:07:06 EDT 2014


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

       via  6d84b6c8ebeaf59a40cd9e4db1a7e897bf76511c (commit)
      from  a0e2de21d7c0cb7d38a3fbfa279e2f8278c5280d (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 6d84b6c8ebeaf59a40cd9e4db1a7e897bf76511c
Author: Tim Pierce <twp at curoverse.com>
Date:   Mon Sep 15 13:06:45 2014 -0400

    3699: delete pi['uuid'] before committing

diff --git a/sdk/python/arvados/commands/copy.py b/sdk/python/arvados/commands/copy.py
index c1d5f12..b250119 100755
--- a/sdk/python/arvados/commands/copy.py
+++ b/sdk/python/arvados/commands/copy.py
@@ -133,7 +133,6 @@ def api_for_instance(instance_name):
 def copy_pipeline_instance(pi_uuid, src, dst, dst_git_repo, dst_project=None, recursive=True):
     # Fetch the pipeline instance record.
     pi = src.pipeline_instances().get(uuid=pi_uuid).execute()
-    pi['properties']['copied_from_pipeline_instance_uuid'] = pi_uuid
 
     if recursive:
         # Copy the pipeline template and save the copied template.
@@ -162,6 +161,8 @@ def copy_pipeline_instance(pi_uuid, src, dst, dst_git_repo, dst_project=None, re
         print >>sys.stderr, "You are responsible for making sure all pipeline dependencies have been updated."
 
     # Create the new pipeline instance at the destination Arvados.
+    pi['properties']['copied_from_pipeline_instance_uuid'] = pi_uuid
+    del pi['uuid']
     new_pi = dst.pipeline_instances().create(body=pi).execute()
     return new_pi
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list