Merge branch '15397-remove-obsolete-apis'

[arvados.git] / sdk / python / arvados / commands / migrate19.py

diff --git a/sdk/python/arvados/commands/migrate19.py b/sdk/python/arvados/commands/migrate19.py
index 3ce47b20660bc68c51833b981cdfdda17c6672e8..7bcc784fd755be1a20a324d5afe099fd0d1213a0 100644 (file)
--- a/sdk/python/arvados/commands/migrate19.py
+++ b/sdk/python/arvados/commands/migrate19.py
@@ -2,8 +2,6 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-from __future__ import print_function
-from __future__ import division
 import argparse
 import time
 import sys
@@ -18,6 +16,7 @@ import arvados
 import arvados.commands.keepdocker
 from arvados._version import __version__
 from arvados.collection import CollectionReader
+from .. import util
 
 logger = logging.getLogger('arvados.migrate-docker19')
 logger.setLevel(logging.DEBUG if arvados.config.get('ARVADOS_DEBUG')
@@ -29,6 +28,7 @@ _migration_link_name = 'migrate_1.9_1.10'
 class MigrationFailed(Exception):
     pass
 
+@util._deprecated('3.0')
 def main(arguments=None):
     """Docker image format migration tool for Arvados.
 
@@ -116,18 +116,23 @@ def main(arguments=None):
         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],
-    ])
+    migration_links = arvados.util.keyset_list_all(
+        api_client.links().list,
+        filters=[
+            ['link_class', '=', _migration_link_class],
+            ['name', '=', _migration_link_name],
+        ],
+        order='uuid')
 
     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", "manifest_text", "owner_uuid"])
+    items = arvados.util.keyset_list_all(
+        api_client.collections().list,
+        filters=[["uuid", "in", [img["collection"] for img in old_images]]],
+        select=["uuid", "portable_data_hash", "manifest_text", "owner_uuid"],
+        order='uuid')
     uuid_to_collection = {i["uuid"]: i for i in items}
 
     need_migrate = {}