[ARVADOS] updated: 3ede45e668eaad4cb23cc11f11a1a7ea6e688c88

[Git user]

Summary of changes:
 sdk/python/arvados/commands/keepdocker.py | 41 ++++++++++++++++---------------
 1 file changed, 21 insertions(+), 20 deletions(-)

       via  3ede45e668eaad4cb23cc11f11a1a7ea6e688c88 (commit)
       via  16ec502827d038e4afe61faae53c64b17e0a0767 (commit)
      from  1535c8d141b07f246a501d288a0d3a903ce41a56 (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 3ede45e668eaad4cb23cc11f11a1a7ea6e688c88
Merge: 1535c8d 16ec502
Author: Brett Smith <brett at curoverse.com>
Date:   Fri Apr 29 17:28:15 2016 -0400

    Merge branch '8963-arv-copy-link-properties-wip'
    
    Closes #8963, #9110.


commit 16ec502827d038e4afe61faae53c64b17e0a0767
Author: Brett Smith <brett at curoverse.com>
Date:   Thu Apr 28 17:32:38 2016 -0400

    8963: arv-keepdocker copies metadata links' properties.

diff --git a/sdk/python/arvados/commands/keepdocker.py b/sdk/python/arvados/commands/keepdocker.py
index 72c5e8d..e8ce2ee 100644
--- a/sdk/python/arvados/commands/keepdocker.py
+++ b/sdk/python/arvados/commands/keepdocker.py
@@ -283,6 +283,9 @@ def list_images_in_arv(api_client, num_retries, image_name=None, image_tag=None)
     return [(image['collection'], image) for image in images
             if image['collection'] in existing_coll_uuids]
 
+def items_owned_by(owner_uuid, arv_items):
+    return (item for item in arv_items if item['owner_uuid'] == owner_uuid)
+
 def main(arguments=None, stdout=sys.stdout):
     args = arg_parser.parse_args(arguments)
     api = arvados.api('v1')
@@ -326,10 +329,10 @@ def main(arguments=None, stdout=sys.stdout):
                 num_retries=args.retries)['uuid']
 
         # Find image hash tags
-        existing_links = api.links().list(
+        existing_links = _get_docker_links(
+            api, args.retries,
             filters=[['link_class', '=', 'docker_image_hash'],
-                     ['name', '=', image_hash]]
-            ).execute(num_retries=args.retries)['items']
+                     ['name', '=', image_hash]])
         if existing_links:
             # get readable collections
             collections = api.collections().list(
@@ -339,21 +342,18 @@ def main(arguments=None, stdout=sys.stdout):
 
             if collections:
                 # check for repo+tag links on these collections
-                existing_repo_tag = (api.links().list(
-                    filters=[['link_class', '=', 'docker_image_repo+tag'],
-                             ['name', '=', image_repo_tag],
-                             ['head_uuid', 'in', collections]]
-                    ).execute(num_retries=args.retries)['items']) if image_repo_tag else []
-
-                # Filter on elements owned by the parent project
-                owned_col = [c for c in collections if c['owner_uuid'] == parent_project_uuid]
-                owned_img = [c for c in existing_links if c['owner_uuid'] == parent_project_uuid]
-                owned_rep = [c for c in existing_repo_tag if c['owner_uuid'] == parent_project_uuid]
-
-                if owned_col:
-                    # already have a collection owned by this project
-                    coll_uuid = owned_col[0]['uuid']
+                if image_repo_tag:
+                    existing_repo_tag = _get_docker_links(
+                        api, args.retries,
+                        filters=[['link_class', '=', 'docker_image_repo+tag'],
+                                 ['name', '=', image_repo_tag],
+                                 ['head_uuid', 'in', collections]])
                 else:
+                    existing_repo_tag = []
+
+                try:
+                    coll_uuid = next(items_owned_by(parent_project_uuid, collections))['uuid']
+                except StopIteration:
                     # create new collection owned by the project
                     coll_uuid = api.collections().create(
                         body={"manifest_text": collections[0]['manifest_text'],
@@ -363,14 +363,15 @@ def main(arguments=None, stdout=sys.stdout):
                         ).execute(num_retries=args.retries)['uuid']
 
                 link_base = {'owner_uuid': parent_project_uuid,
-                             'head_uuid':  coll_uuid }
+                             'head_uuid':  coll_uuid,
+                             'properties': existing_links[0]['properties']}
 
-                if not owned_img:
+                if not any(items_owned_by(parent_project_uuid, existing_links)):
                     # create image link owned by the project
                     make_link(api, args.retries,
                               'docker_image_hash', image_hash, **link_base)
 
-                if not owned_rep and image_repo_tag:
+                if image_repo_tag and not any(items_owned_by(parent_project_uuid, existing_repo_tag)):
                     # create repo+tag link owned by the project
                     make_link(api, args.retries, 'docker_image_repo+tag',
                               image_repo_tag, **link_base)

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


hooks/post-receive
--