Add extra unit test case for more than 1 ephemeral disk in BDM

Bug #1317321 reported by Thang Pham
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Thang Pham

Bug Description

There was a comment on https://review.openstack.org/#/c/90583/ (after it was approved for merge) to add an extra test case to handle more than 1 ephemeral disk, as well as correct a comment in the code to be more accurate. The purpose of this bug is to add the extra test case for _get_instance_block_device_info to test more than 1 ephemeral disk and correct one of the comments in _get_instance_block_device_info.

Thang Pham (thang-pham)
Changed in nova:
assignee: nobody → Thang Pham (thang-pham)
importance: Undecided → Low
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

Reviewed: https://review.openstack.org/92754
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=524ee4913d9015c2893cca69cefbc80fbe6878bb
Submitter: Jenkins
Branch: master

commit 524ee4913d9015c2893cca69cefbc80fbe6878bb
Author: Thang Pham <email address hidden>
Date: Wed May 7 22:17:29 2014 -0400

    Unit test case for more than 1 ephemeral disks in BDM

    The following patch modifies the test case for
    _get_instance_block_device_info to handle more than 1 ephemeral
    disks in the BDM. This is to address one of the comments made
    in https://review.openstack.org/#/c/90583/ after it was already
    approved for merger. This patch also corrects a comment in
    _get_instance_block_device_info to be more accurate.

    Change-Id: I11e97f2e2ef999fa08af1693483d56bf8ad0e556
    Closes-Bug: #1317321

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-1 → 2014.2
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.