volume type test uses non-existing capabilities as type extra specs

Bug #1098017 reported by Huang Zhiteng
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Critical
Huang Zhiteng

Bug Description

In Cinder's G2 release, a new scheduler called filter scheduler has been included. This scheduler decides volume placement based on filtering result with various filters. One of the default filter is 'CapabilitiesFilter' which takes volume type extra specs as capabilities requirement, and would check the capabilities reported by volume backends. This new change impact the volume type test in tempest https://github.com/openstack/tempest/blob/master/tempest/tests/volume/admin/test_volume_types.py

The test is to create a volume type with extra specs and then create/delete volume using this type. It worked well before when simple scheduler was the default, which took no use of volume type and extra spec info. But it would fail when filter scheduler is used. The reason is that no volume driver supports the fake capabilities in the test case. So scheduler fails to find one host to serve that request. It's like creating a Nova flavor stating requires CPU arch to be SPARC while running the test on top X86 servers.

This test has been show stopper for tempest gating job if Cinder uses filter scheduler as default. The fix is simple, use real capabilities instead of fake ones in test.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
assignee: nobody → Huang Zhiteng (zhiteng-huang)
status: New → In Progress
Revision history for this message
Jay Pipes (jaypipes) wrote :

Thanks for looking into this, Huang!

Changed in tempest:
importance: Undecided → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/19359
Committed: http://github.com/openstack/tempest/commit/631df481c8050597f17247f795ee649271b7d849
Submitter: Jenkins
Branch: master

commit 631df481c8050597f17247f795ee649271b7d849
Author: Zhiteng Huang <email address hidden>
Date: Thu Jan 10 10:56:57 2013 +0800

    Use real capabilites for volume type instead of fake ones

    Replace fake capabilities requirements in volume type extra spec
    with real capabilites that Cinder/iSCSI driver would report.

    fix bug: 1098017

    Change-Id: I0e226c14df95097cfdf653f0a13a45b8acff39da

Changed in tempest:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/21073
Committed: http://github.com/openstack/tempest/commit/c3d654d92fa47e34f2b9c93a8f75633cae5902fd
Submitter: Jenkins
Branch: master

commit c3d654d92fa47e34f2b9c93a8f75633cae5902fd
Author: Zhiteng Huang <email address hidden>
Date: Mon Feb 4 02:12:33 2013 +0800

    Use real capabilites for all volume type instead of fake ones

    Replace fake capabilities requirements in all volume type extra
    spec with real capabilites that Cinder/iSCSI driver would report.

    Previous fix in I0e226c14df95097cfdf653f0a13a45b8acff39da isn't
    complete, this change should cover all.

    fix bug: 1098017

    Change-Id: Ie59b11de5e8673e0414b08c5570a14f5cf69a246

Sean Dague (sdague)
Changed in tempest:
milestone: none → havana-3
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.