From 2bbef9ca44650e982da0b4923e32188ab633b72c Mon Sep 17 00:00:00 2001 From: Isaku Yamahata <isaku.yamahata@intel.com> Date: Thu, 7 Aug 2014 14:52:55 +0900 Subject: [PATCH] rename --kwargs to --attributes As discussed at api review, attributes is preferred to kwargs Change-Id: I8d3c6ed8f25ddea44d86f4b40c7172d8fa9be103 --- tackerclient/tacker/v1_0/vm/device.py | 36 +++++++++++++-------------- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/tackerclient/tacker/v1_0/vm/device.py b/tackerclient/tacker/v1_0/vm/device.py index efb7a8ef..cdade23b 100644 --- a/tackerclient/tacker/v1_0/vm/device.py +++ b/tackerclient/tacker/v1_0/vm/device.py @@ -53,10 +53,10 @@ class CreateDevice(tackerV10.CreateCommand): required=True, help='device template id to create device based on') parser.add_argument( - '--kwargs', + '--attributes', metavar='<key>=<value>', action='append', - dest='kwargs', + dest='attributes', default=[], help='instance specific argument') parser.add_argument( @@ -75,16 +75,16 @@ class CreateDevice(tackerV10.CreateCommand): 'template_id': parsed_args.device_template_id, } } - if parsed_args.kwargs: + if parsed_args.attributes: try: - kwargs = dict(key_value.split('=', 1) - for key_value in parsed_args.kwargs) + attributes = dict(key_value.split('=', 1) + for key_value in parsed_args.attributes) except ValueError: - msg = (_('invalid argument for --kwargs %s') % - parsed_args.kwargs) + msg = (_('invalid argument for --attributes %s') % + parsed_args.attributes) raise exceptions.TackerCLIError(msg) - if kwargs: - body[self.resource]['kwargs'] = kwargs + if attributes: + body[self.resource]['attributes'] = attributes if parsed_args.service_context: try: service_context = [dict( @@ -111,25 +111,25 @@ class UpdateDevice(tackerV10.UpdateCommand): def add_known_arguments(self, parser): parser.add_argument( - '--kwargs', + '--attributes', metavar='<key>=<value>', action='append', - dest='kwargs', + dest='attributes', default=[], help='instance specific argument') def args2body(self, parsed_args): body = {self.resource: {}} - if parsed_args.kwargs: + if parsed_args.attributes: try: - kwargs = dict(key_value.split('=', 1) - for key_value in parsed_args.kwargs) + attributes = dict(key_value.split('=', 1) + for key_value in parsed_args.attributes) except ValueError: - msg = (_('invalid argument for --kwargs %s') % - parsed_args.kwargs) + msg = (_('invalid argument for --attributes %s') % + parsed_args.attributes) raise exceptions.TackerCLIError(msg) - if kwargs: - body[self.resource]['kwargs'] = kwargs + if attributes: + body[self.resource]['attributes'] = attributes tackerV10.update_dict(parsed_args, body[self.resource], ['tenant_id']) return body