[ARVADOS] updated: 2accec1f7ca3e2dd1e7fe9ee28bb6a8a6cb74abc

Git user git at public.curoverse.com
Fri Mar 17 19:30:57 EDT 2017


Summary of changes:
 sdk/python/arvados/commands/keepdocker.py | 134 ------------------------------
 sdk/python/bin/arv-migrate-docker19       |   4 +-
 2 files changed, 2 insertions(+), 136 deletions(-)

       via  2accec1f7ca3e2dd1e7fe9ee28bb6a8a6cb74abc (commit)
      from  54837bcae94d30f37f19e4ee436c2bb96bd8f21a (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 2accec1f7ca3e2dd1e7fe9ee28bb6a8a6cb74abc
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Fri Mar 17 17:54:11 2017 -0400

    8567: Better command line UI

diff --git a/sdk/python/arvados/commands/keepdocker.py b/sdk/python/arvados/commands/keepdocker.py
index bbec030..660b45d 100644
--- a/sdk/python/arvados/commands/keepdocker.py
+++ b/sdk/python/arvados/commands/keepdocker.py
@@ -338,140 +338,6 @@ def _uuid2pdh(api, uuid):
         select=['portable_data_hash'],
     ).execute()['items'][0]['portable_data_hash']
 
-_migration_link_class = 'docker_image_migration'
-_migration_link_name = 'migrate_1.9_1.10'
-
-def migrate19():
-    """Docker image format migration tool for Arvados.
-
-    This converts Docker images stored in Arvados from image format v1
-    (Docker <= 1.9) to image format v2 (Docker >= 1.10).
-
-    Requires Docker running on the local host.
-
-    Usage:
-
-    1) Run arvados/docker/migrate-docker19/build.sh to create
-    arvados/migrate-docker19 Docker image.
-
-    2) Set ARVADOS_API_HOST and ARVADOS_API_TOKEN to the cluster you want to migrate.
-
-    3) Run arv-migrate-docker19 from the Arvados Python SDK on the host (not in a container).
-
-    This will query Arvados for v1 format Docker images.  For each image that
-    does not already have a corresponding v2 format image (as indicated by a
-    docker_image_migration tag) it will perform the following process:
-
-    i) download the image from Arvados
-    ii) load it into Docker
-    iii) update the Docker version, which updates the image
-    iv) save the v2 format image and upload to Arvados
-    v) create a migration link
-
-    """
-
-    api_client  = arvados.api()
-
-    user = api_client.users().current().execute()
-    if not user['is_admin']:
-        raise Exception("This command requires an admin token")
-    sys_uuid = user['uuid'][:12] + '000000000000000'
-
-    images = arvados.commands.keepdocker.list_images_in_arv(api_client, 3)
-
-    is_new = lambda img: img['dockerhash'].startswith('sha256:')
-
-    count_new = 0
-    old_images = []
-    for uuid, img in images:
-        if img["dockerhash"].startswith("sha256:"):
-            continue
-        key = (img["repo"], img["tag"], img["timestamp"])
-        old_images.append(img)
-
-    migration_links = arvados.util.list_all(api_client.links().list, filters=[
-        ['link_class', '=', _migration_link_class],
-        ['name', '=', _migration_link_name],
-    ])
-
-    already_migrated = set()
-    for m in migration_links:
-        already_migrated.add(m["tail_uuid"])
-
-    items = arvados.util.list_all(api_client.collections().list,
-                                  filters=[["uuid", "in", [img["collection"] for img in old_images]]],
-                                  select=["uuid", "portable_data_hash"])
-    uuid_to_pdh = {i["uuid"]: i["portable_data_hash"] for i in items}
-    need_migrate = [img for img in old_images
-                    if uuid_to_pdh[img["collection"]] not in already_migrated]
-
-    logger.info("Already migrated %i images", len(already_migrated))
-    logger.info("Need to migrate %i images", len(need_migrate))
-
-    success = []
-    failures = []
-    for old_image in need_migrate:
-        if uuid_to_pdh[old_image["collection"]] in already_migrated:
-            continue
-
-        logger.info("Migrating %s:%s (%s)", old_image["repo"], old_image["tag"], old_image["collection"])
-
-        oldcol = CollectionReader(old_image["collection"])
-        tarfile = oldcol.keys()[0]
-
-        try:
-            varlibdocker = tempfile.mkdtemp()
-            with tempfile.NamedTemporaryFile() as envfile:
-                envfile.write("ARVADOS_API_HOST=%s\n" % (os.environ["ARVADOS_API_HOST"]))
-                envfile.write("ARVADOS_API_TOKEN=%s\n" % (os.environ["ARVADOS_API_TOKEN"]))
-                if "ARVADOS_API_HOST_INSECURE" in os.environ:
-                    envfile.write("ARVADOS_API_HOST_INSECURE=%s\n" % (os.environ["ARVADOS_API_HOST_INSECURE"]))
-                envfile.flush()
-
-                dockercmd = ["docker", "run",
-                             "--privileged",
-                             "--rm",
-                             "--env-file", envfile.name,
-                             "--volume", "%s:/var/lib/docker" % varlibdocker,
-                             "arvados/migrate-docker19",
-                             "/root/migrate.sh",
-                             "%s/%s" % (old_image["collection"], tarfile),
-                             tarfile[0:40],
-                             old_image["repo"],
-                             old_image["tag"],
-                             oldcol.api_response()["owner_uuid"]]
-
-                out = subprocess.check_output(dockercmd)
-
-            migrated = re.search(r"Migrated uuid is ([a-z0-9]{5}-[a-z0-9]{5}-[a-z0-9]{15})", out)
-            if migrated:
-                newcol = CollectionReader(migrated.group(1))
-
-                api_client.links().create(body={"link": {
-                    'owner_uuid': sys_uuid,
-                    'link_class': arvados.commands.keepdocker._migration_link_class,
-                    'name': arvados.commands.keepdocker._migration_link_name,
-                    'tail_uuid': oldcol.portable_data_hash(),
-                    'head_uuid': newcol.portable_data_hash()
-                    }}).execute(num_retries=3)
-
-                logger.info("Migrated '%s' to '%s'", oldcol.portable_data_hash(), newcol.portable_data_hash())
-                already_migrated.add(oldcol.portable_data_hash())
-                success.append(old_image["collection"])
-            else:
-                logger.error("Error migrating '%s'", old_image["collection"])
-                failures.append(old_image["collection"])
-        except Exception as e:
-            logger.exception("Migration failed")
-            failures.append(old_image["collection"])
-        finally:
-            shutil.rmtree(varlibdocker)
-
-    logger.info("Successfully migrated %i images", len(success))
-    if failures:
-        logger.error("Failure migrating images: %s", failures)
-
-
 def main(arguments=None, stdout=sys.stdout):
     args = arg_parser.parse_args(arguments)
     api = arvados.api('v1')
diff --git a/sdk/python/bin/arv-migrate-docker19 b/sdk/python/bin/arv-migrate-docker19
index fcef8d7..8d928a1 100755
--- a/sdk/python/bin/arv-migrate-docker19
+++ b/sdk/python/bin/arv-migrate-docker19
@@ -1,4 +1,4 @@
 #!/usr/bin/env python
 
-from arvados.commands.keepdocker import migrate19
-migrate19()
+from arvados.commands.migrate19 import main
+main()

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list