Cinder V2 API now returns a 202 on create, but we haven't updated Tempest to handle this

Bug #1500923 reported by John Griffith
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
tempest
Fix Released
High
John Griffith

Bug Description

Default in the gate for Cinder API is finally V2, one of the changes that was discovered is the response code from create. Need to handle/expect 202 from create calls in some tests.

Revision history for this message
John Griffith (john-griffith) wrote :
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/229014

Changed in tempest:
assignee: nobody → John Griffith (john-griffith)
status: New → In Progress
Matt Riedemann (mriedem)
Changed in tempest:
importance: Undecided → High
Revision history for this message
Matthew Treinish (treinish) wrote :

The issue here was introduced by https://github.com/openstack/tempest/commit/8843f0f0768019c76be72b4be2f6a156cdbe3d78

the changes to tempest.common.compute @ L90 to create a volume is unconditionally using the v1 client. We just need to change that to have an if else like:

https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L92

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

Reviewed: https://review.openstack.org/229014
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=bc678ad48ed3b389673262775c33ec89e4f2369c
Submitter: Jenkins
Branch: master

commit bc678ad48ed3b389673262775c33ec89e4f2369c
Author: John Griffith <email address hidden>
Date: Tue Sep 29 09:38:39 2015 -0600

    Check cinder api version in compute tests

    We've turned on Cinder V2 API as default so a few things
    have changed. One of those things is the response code
    for volume/snapshot create. These no longer return a
    200, but instead a 202.

    The result is that some tests are hard coded to still
    use the default cinder client API version. This
    means that those tests will receive the wrong response
    code.

    This patch just adds a check to common/compute.py
    where we setup for the volume backed migration test.

    There is probably a better solution to do this further
    up and propogate it everywhere, but this provides a
    quick fix for the one know case that fails.

    Change-Id: I3c1524ade051a8ace1e393085efb1d24fc832c9b
    Closes-Bug: #1500923

Changed in tempest:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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