Fix ListOpt to trim whitespace

- throw in an extra strip() in the list parsing code
- add a test case to verify that it works!

Fixes LP #1079299

Change-Id: I4f0864c72ecd2569d0461c301acda395c87a93e0

2 files changed, 5 insertions, 1 deletions

diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py
index e967c61..c8a15d7 100644
--- a/openstack/common/cfg.py
+++ b/openstack/common/cfg.py
@@ -745,7 +745,7 @@ class ListOpt(Opt):
         """
         def __call__(self, parser, namespace, values, option_string=None):
             if values is not None:
-                values = values.split(',')
+                values = [a.strip() for a in values.split(',')]
             setattr(namespace, self.dest, values)
 
     def _get_from_config_parser(self, cparser, section):
diff --git a/tests/unit/test_cfg.py b/tests/unit/test_cfg.py
index 384a57a..bd9e914 100644
--- a/tests/unit/test_cfg.py
+++ b/tests/unit/test_cfg.py
@@ -219,6 +219,10 @@ class CliOptsTestCase(BaseTestCase):
         self._do_cli_test(ListOpt, None,
                           ['--foo', 'blaa,bar'], ['blaa', 'bar'])
 
+    def test_list_arg_with_spaces(self):
+        self._do_cli_test(ListOpt, None,
+                          ['--foo', 'blaa ,bar'], ['blaa', 'bar'])
+
     def test_list_arg_deprecated(self):
         self._do_cli_test(ListOpt, None,
                           ['--oldfoo', 'blaa,bar'], ['blaa', 'bar'])