Improve admin CLI result output (#91)

* Prettify the console output for admin CLI commands
* Update changelog

Author: achimnol

CHANGELOG.rst

@@ -7,6 +7,8 @@ Changes
 * NEW: Add ``-f`` / ``--forced`` option to ``backend.ai terminate`` command
   and add an optional ``forced`` keyword-argument to SDK API (``Kernel.destroy()``). (#89)
 
+* IMPROVE: Prettify the console output for admin CLI commands. (#91)
+
 19.09.5 (2020-03-08)
 --------------------
 

src/ai/backend/client/cli/admin/images.py

@@ -4,7 +4,7 @@
 
 from . import admin
 from ...session import Session
-from ..pretty import print_error
+from ..pretty import print_done, print_warn, print_fail, print_error
 
 
 @admin.command()
@@ -29,7 +29,7 @@ def images(operation):
             print_error(e)
             sys.exit(1)
         if len(items) == 0:
-            print('There are no registered images.')
+            print_warn('There are no registered images.')
             return
         print(tabulate((item.values() for item in items),
                        headers=(item[0] for item in fields),
@@ -49,9 +49,9 @@ def rescan_images(registry):
             print_error(e)
             sys.exit(1)
         if result['ok']:
-            print("kernel image metadata updated")
+            print_done("Updated the image metadata from the configured registries.")
         else:
-            print("rescanning failed: {0}".format(result['msg']))
+            print_fail(f"Rescanning has failed: {result['msg']}")
 
 
 @admin.command()
@@ -66,9 +66,9 @@ def alias_image(alias, target):
             print_error(e)
             sys.exit(1)
         if result['ok']:
-            print("alias {0} created for target {1}".format(alias, target))
+            print_done(f"An alias has created: {alias} -> {target}")
         else:
-            print(result['msg'])
+            print_fail(f"Aliasing has failed: {result['msg']}")
 
 
 @admin.command()
@@ -82,6 +82,6 @@ def dealias_image(alias):
             print_error(e)
             sys.exit(1)
         if result['ok']:
-            print("alias {0} removed.".format(alias))
+            print_done(f"The alias has been removed: {alias}")
         else:
-            print(result['msg'])
+            print_fail(f"Dealiasing has failed: {result['msg']}")

src/ai/backend/client/cli/admin/keypairs.py

@@ -5,7 +5,7 @@
 
 from . import admin
 from ...session import Session
-from ..pretty import print_error, print_fail
+from ..pretty import print_done, print_warn, print_error, print_fail
 
 
 @admin.command()
@@ -79,8 +79,8 @@ def keypairs(ctx, user_id, is_active):
             print_error(e)
             sys.exit(1)
         if len(items) == 0:
-            print('There are no matching keypairs associated '
-                  'with the user ID {0}'.format(user_id))
+            print_warn('There are no matching keypairs associated '
+                       'with the user ID {0}'.format(user_id))
             return
         print(tabulate((item.values() for item in items),
                        headers=(item[0] for item in fields)))
@@ -151,7 +151,7 @@ def update(access_key, resource_policy, is_admin, is_active,  rate_limit):
         if not data['ok']:
             print_fail('KeyPair update has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Key pair is updated: ' + access_key + '.')
+        print_done('Key pair is updated: ' + access_key + '.')
 
 
 @keypairs.command()
@@ -171,7 +171,7 @@ def delete(access_key):
         if not data['ok']:
             print_fail('KeyPair deletion has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Key pair is deleted: ' + access_key + '.')
+        print_done('Key pair is deleted: ' + access_key + '.')
 
 
 @keypairs.command()
@@ -191,7 +191,7 @@ def activate(access_key):
         if not data['ok']:
             print_fail('KeyPair activation has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Key pair is activated: ' + access_key + '.')
+        print_done('Key pair is activated: ' + access_key + '.')
 
 
 @keypairs.command()
@@ -211,4 +211,4 @@ def deactivate(access_key):
         if not data['ok']:
             print_fail('KeyPair deactivation has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Key pair is deactivated: ' + access_key + '.')
+        print_done('Key pair is deactivated: ' + access_key + '.')

src/ai/backend/client/cli/admin/resource_policies.py

@@ -5,7 +5,7 @@
 
 from . import admin
 from ...session import Session
-from ..pretty import print_error, print_fail
+from ..pretty import print_done, print_warn, print_error, print_fail
 
 
 @admin.command()
@@ -37,7 +37,7 @@ def keypair_resource_policy(name):
             sys.exit(1)
         rows = []
         if info is None:
-            print('No such resource policy.')
+            print_warn('No such resource policy.')
             sys.exit(1)
         for name, key in fields:
             rows.append((name, info[key]))
@@ -72,7 +72,7 @@ def keypair_resource_policies(ctx):
             print_error(e)
             sys.exit(1)
         if len(items) == 0:
-            print('There are no keypair resource policies.')
+            print_warn('There are no keypair resource policies.')
             return
         print(tabulate((item.values() for item in items),
                        headers=(item[0] for item in fields)))
@@ -129,7 +129,7 @@ def add(name, default_for_unspecified, total_resource_slots, max_concurrent_sess
                        .format(data['msg']))
             sys.exit(1)
         item = data['resource_policy']
-        print('Keypair resource policy ' + item['name'] + ' is created.')
+        print_done('Keypair resource policy ' + item['name'] + ' is created.')
 
 
 @keypair_resource_policies.command()
@@ -179,7 +179,7 @@ def update(name, default_for_unspecified, total_resource_slots,
             print_fail('KeyPair Resource Policy creation has failed: {0}'
                        .format(data['msg']))
             sys.exit(1)
-        print('Update succeeded.')
+        print_done('Update succeeded.')
 
 
 @keypair_resource_policies.command()
@@ -203,4 +203,4 @@ def delete(name):
             print_fail('KeyPair Resource Policy deletion has failed: {0}'
                        .format(data['msg']))
             sys.exit(1)
-        print('Resource policy ' + name + ' is deleted.')
+        print_done('Resource policy ' + name + ' is deleted.')

src/ai/backend/client/cli/admin/scaling_groups.py

@@ -4,7 +4,7 @@
 from tabulate import tabulate
 
 from . import admin
-from ..pretty import print_error, print_fail
+from ..pretty import print_done, print_warn, print_error, print_fail
 from ...session import Session
 
 
@@ -18,7 +18,7 @@ def list_scaling_groups(group):
             print_error(e)
             sys.exit(1)
         if len(resp) < 1:
-            print('There is no scaling group available.')
+            print_warn('There is no scaling group available.')
             return
         print(resp)
 
@@ -127,7 +127,7 @@ def add(name, description, inactive,
             print_fail('Scaling group creation has failed: {0}'.format(data['msg']))
             sys.exit(1)
         item = data['scaling_group']
-        print('Scaling group name {0} is created.'.format(item['name']))
+        print_done('Scaling group name {0} is created.'.format(item['name']))
 
 
 @scaling_groups.command()
@@ -168,7 +168,7 @@ def update(name, description, inactive,
         if not data['ok']:
             print_fail('Scaling group update has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Scaling group {0} is updated.'.format(name))
+        print_done('Scaling group {0} is updated.'.format(name))
 
 
 @scaling_groups.command()
@@ -188,7 +188,7 @@ def delete(name):
         if not data['ok']:
             print_fail('Scaling group deletion has failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Scaling group is deleted: ' + name + '.')
+        print_done('Scaling group is deleted: ' + name + '.')
 
 
 @scaling_groups.command()
@@ -211,7 +211,7 @@ def associate_scaling_group(scaling_group, domain):
         if not data['ok']:
             print_fail('Associating scaling group with domain failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Scaling group {} is assocatiated with domain {}.'.format(scaling_group, domain))
+        print_done('Scaling group {} is assocatiated with domain {}.'.format(scaling_group, domain))
 
 
 @scaling_groups.command()
@@ -234,4 +234,4 @@ def dissociate_scaling_group(scaling_group, domain):
         if not data['ok']:
             print_fail('Dissociating scaling group from domain failed: {0}'.format(data['msg']))
             sys.exit(1)
-        print('Scaling group {} is dissociated from domain {}.'.format(scaling_group, domain))
+        print_done('Scaling group {} is dissociated from domain {}.'.format(scaling_group, domain))