From b8d72cee27dd81b5749c64945475ea8781cb5fdf Mon Sep 17 00:00:00 2001 From: Jas Date: Mon, 29 Feb 2016 16:06:05 -0600 Subject: [PATCH] fix: Exception message includes unnecessary class args Fix misusages of ArgumentTypeError which causes a tuple of class instance and error message string to be printed rather than just the error message string itsself. Change-Id: I0e997f86bb6603930cc92e90efcb48155f62ffb5 Closes-bug: #1551426 --- openstackclient/common/parseractions.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openstackclient/common/parseractions.py b/openstackclient/common/parseractions.py index 7d332a5f5b..c30058c8e7 100644 --- a/openstackclient/common/parseractions.py +++ b/openstackclient/common/parseractions.py @@ -89,7 +89,7 @@ class MultiKeyValueAction(argparse.Action): else: msg = ("Expected key=value pairs separated by comma, " "but got: %s" % (str(kv))) - raise argparse.ArgumentTypeError(self, msg) + raise argparse.ArgumentTypeError(msg) # Check key validation valid_keys = self.required_keys | self.optional_keys @@ -160,4 +160,4 @@ class NonNegativeAction(argparse.Action): setattr(namespace, self.dest, values) except Exception: msg = "%s expected a non-negative integer" % (str(option_string)) - raise argparse.ArgumentTypeError(self, msg) + raise argparse.ArgumentTypeError(msg)