diff --git a/watcher/conf/placement_client.py b/watcher/conf/placement_client.py index 2afb887fc..81d89a6e8 100644 --- a/watcher/conf/placement_client.py +++ b/watcher/conf/placement_client.py @@ -38,4 +38,4 @@ def register_opts(conf): def list_opts(): - return [(placement_group.name, placement_opts)] + return [(placement_group, placement_opts)] diff --git a/watcher/tests/conf/test_list_opts.py b/watcher/tests/conf/test_list_opts.py index f6a63d410..fb478c83e 100755 --- a/watcher/tests/conf/test_list_opts.py +++ b/watcher/tests/conf/test_list_opts.py @@ -16,6 +16,7 @@ # limitations under the License. import mock +from oslo_config import cfg from stevedore import extension from watcher.conf import opts @@ -74,7 +75,10 @@ class TestListOpts(base.TestCase): result = opts.list_opts() self.assertIsNotNone(result) - for section_name, options in result: + for name_or_group, options in result: + section_name = name_or_group + if isinstance(name_or_group, cfg.OptGroup): + section_name = name_or_group.name self.assertIn(section_name, expected_sections) self.assertTrue(len(options)) @@ -106,7 +110,10 @@ class TestListOpts(base.TestCase): result = opts.list_opts() self.assertIsNotNone(result) - for section_name, options in result: + for name_or_group, options in result: + section_name = name_or_group + if isinstance(name_or_group, cfg.OptGroup): + section_name = name_or_group.name self.assertIn(section_name, expected_sections) self.assertTrue(len(options))