Reuse already registerd oslo config groups in tempest plugin

Bug #1626575 reported by Nishant Kumar
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Nishant Kumar

Bug Description

This bug addresses two of the config groups that are being created in tempest plugin for cinder. Instead of
registering new groups we can reuse already registered groups in tempest/config.py.

Changes should be made to the following groups:

a) service_available_group - We don't need this group in plugin code because it is already being registered in tempest code and service availability is being checked for in the /tempest/api/volume/base.py and the test cases in plugin code inherits this base.py

b) cinder group - Instead of registering a new cinder group for plugin tests we can add plugin specific configuration options in the existing volume group as the same conf object from tempest is being passed on to the plugin.

Changed in cinder:
assignee: nobody → Nishant Kumar (nishant-tech07)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/375808

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/375808
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=4e1643ea5b13867c0c5adebca26112a8c804e4b6
Submitter: Jenkins
Branch: master

commit 4e1643ea5b13867c0c5adebca26112a8c804e4b6
Author: Nishant Kumar <email address hidden>
Date: Fri Sep 23 20:11:58 2016 +0530

    Reuse already existing groups in tempest plugin

    Instead of registering new groups we can reuse already registered
    groups in tempest/config.py.

    Changes should be made to the following groups:

    a) service_available_group - We don't need this group in plugin code
    because it is already being registered in tempest code and service
    availability is being checked for in the /tempest/api/volume/base.py
    and the test cases in plugin code inherits this base.py

    b) cinder group - Instead of registering a new cinder group for plugin
    tests we can add plugin specific configuration options in the existing
    volume group as the same conf object from tempest is being passed on
    to the plugin

    Change-Id: Ie0ca52ec389c581ca47db3e9bbb0793a2bb47fdf
    Closes-Bug: #1626575

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 10.0.0.0b3

This issue was fixed in the openstack/cinder 10.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.