openstack
/
murano-tempest-plugin
Code Issues Proposed changes

Make use of already registered group 

service_available_group - We don't need this group in plugin
code because it is already being registered in tempest config
and since we are using the same config object from tempest,
we can make use of this group directly here instead of using a
new group with the same name.

closes-Bug: #1661213

Change-Id: Iabd4a7c356b1f454585054218bf07801e0339bde
This commit is contained in:
PRAMOD KUMAR SINGH 2017-02-02 17:33:50 +05:30
parent adab70409a
commit d3d4418cb1
2 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
murano_tempest_tests/config.py
View File

@ -15,10 +15,7 @@
from oslo_config import cfg
service_available_group = cfg.OptGroup(name="service_available",
title="Available OpenStack Services")
ServiceAvailableGroup = [
service_option = [
cfg.BoolOpt("murano",
default=True,
help="Whether or not murano is expected to be available"),

8
murano_tempest_tests/plugin.py
View File

@ -31,8 +31,8 @@ class MuranoTempestPlugin(plugins.TempestPlugin):
def register_opts(self, conf):
config.register_opt_group(
conf, config_application_catalog.service_available_group,
config_application_catalog.ServiceAvailableGroup)
conf, config.service_available_group,
config_application_catalog.service_option)
config.register_opt_group(
conf, config_application_catalog.application_catalog_group,
config_application_catalog.ApplicationCatalogGroup)
@ -48,7 +48,7 @@ class MuranoTempestPlugin(plugins.TempestPlugin):
config_application_catalog.ApplicationCatalogGroup),
(config_application_catalog.service_broker_group.name,
config_application_catalog.ServiceBrokerGroup),
('service_available',
config_application_catalog.ServiceAvailableGroup),
(config.service_available_group.name,
config_application_catalog.service_option),
(config_application_catalog.artifacts_group.name,
config_application_catalog.ArtifactsGroup)]