commit 02278309dd2585299d45578ccaca821a6ddcd684
author Nishant Kumar <nishant.e.kumar@ericsson.com> Sat Oct 01 12:50:39 2016 +0530
committer Nishant Kumar <nishant.e.kumar@ericsson.com> Tue Oct 04 20:38:15 2016 +0530
tree 2c9bb6d305d261b658838bfd571a3b77374a675f
parent 35d82e59aa45e192fa81d11f7507225a0001714d

Stop adding ServiceAvailable group option

Service available group already exists.Therefore we don't need to
register this group here again.

Change-Id: I9680abe0a177a80860ec6ba1dbacc04fac41758d
Closes-Bug: #1621036

manila_tempest_tests/config.py

diff --git a/manila_tempest_tests/config.py b/manila_tempest_tests/config.py
index f0c8dc7..1c1735d 100644
--- a/manila_tempest_tests/config.py
+++ b/manila_tempest_tests/config.py
@@ -17,14 +17,10 @@
 
 from oslo_config import cfg
 
-service_available_group = cfg.OptGroup(name="service_available",
-                                       title="Available OpenStack Services")
-
-ServiceAvailableGroup = [
-    cfg.BoolOpt("manila",
-                default=True,
-                help="Whether or not manila is expected to be available"),
-]
+service_option = cfg.BoolOpt("manila",
+                             default=True,
+                             help="Whether or not manila is expected to be "
+                                  "available")
 
 share_group = cfg.OptGroup(name="share", title="Share Service Options")
 

manila_tempest_tests/plugin.py

diff --git a/manila_tempest_tests/plugin.py b/manila_tempest_tests/plugin.py
index 02c3c98..0a97747 100644
--- a/manila_tempest_tests/plugin.py
+++ b/manila_tempest_tests/plugin.py
@@ -16,7 +16,6 @@
 
 import os
 
-from tempest import config
 from tempest.test_discover import plugins
 
 from manila_tempest_tests import config as config_share
@@ -31,11 +30,10 @@
         return full_test_dir, base_path
 
     def register_opts(self, conf):
-        config.register_opt_group(
-            conf, config_share.service_available_group,
-            config_share.ServiceAvailableGroup)
-        config.register_opt_group(conf, config_share.share_group,
-                                  config_share.ShareGroup)
+        conf.register_opt(config_share.service_option,
+                          group='service_available')
+        conf.register_group(config_share.share_group)
+        conf.register_opts(config_share.ShareGroup, group='share')
 
         # NOTE(vponomaryov): set opt 'capability_snapshot_support' by
         # default equal to opt 'run_snapshot_tests'.
@@ -48,4 +46,4 @@
 
     def get_opt_lists(self):
         return [(config_share.share_group.name, config_share.ShareGroup),
-                ('service_available', config_share.ServiceAvailableGroup)]
+                ('service_available', [config_share.service_option])]