The index of lun ID is wrong in test

Bug #1225884 reported by zhangchao
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
zhangchao

Bug Description

The index of lun ID should be "params[params.index('-lun')] + 1 " , rather than "params[params.index('-lun')]".

zhangchao (zhangchao010)
Changed in cinder:
assignee: nobody → zhangchao (zhangchao010)
status: New → In Progress
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/46699

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

Reviewed: https://review.openstack.org/46699
Committed: http://github.com/openstack/cinder/commit/306dc1792dba21e016d319c4a0211e9d4dcc3ad9
Submitter: Jenkins
Branch: master

commit 306dc1792dba21e016d319c4a0211e9d4dcc3ad9
Author: zhangchao010 <email address hidden>
Date: Tue Sep 17 23:53:24 2013 +0800

    Fix huawei driver test issues

    Replace "params[params.index('-lun')]" with
    "params[params.index('-lun') + 1]" to get lun ID correctly.
    Replace the invalid controller ID "N" with "B".
    Dict CLONED_LUN_INFO has no key provider_location, delete it.

    Moreover, simplify the "if..., elif..." module with "getattr"
    method in fake recv function.

    fixes bug 1225884
    Change-Id: Ic4cbf8b07321cdecd6290e8cd90bca1b09dca7fd

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-rc1 → 2013.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.