From 33b2830e9c81f87fb37e51eca65f6e98143d1e06 Mon Sep 17 00:00:00 2001 From: Sulochan Acharya Date: Thu, 1 Nov 2012 17:43:39 -0500 Subject: [PATCH] Allows deletion of multiple servers through CLI Adds nargs to argparse for delete command to allow multiple (optional) positional server argument. This allows users to delete multiple servers using nova delete. For example: nova delete xxx yyy aaa zzz will delete valid servers xxx, yyy and zzz and print exception msg for invalid delete of server aaa. Fixes bug892061 Change-Id: I2abe6329e489bfbd3e5ae699e97ef098b38c4537 --- novaclient/v1_1/shell.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/novaclient/v1_1/shell.py b/novaclient/v1_1/shell.py index 516f21e59..1ce6a4460 100644 --- a/novaclient/v1_1/shell.py +++ b/novaclient/v1_1/shell.py @@ -1011,10 +1011,15 @@ def do_show(cs, args): _print_server(cs, args) -@utils.arg('server', metavar='', help='Name or ID of server.') +@utils.arg('server', metavar='', nargs='+', + help='Name or ID of server(s).') def do_delete(cs, args): - """Immediately shut down and delete a server.""" - _find_server(cs, args.server).delete() + """Immediately shut down and delete specified server(s).""" + for server in args.server: + try: + _find_server(cs, server).delete() + except Exception, e: + print e def _find_server(cs, server):