The index of lun ID is wrong in test

Bug #1225884 reported by zhangchao on 2013-09-16
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
zhangchao

Bug Description

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

zhangchao (zhangchao010) on 2013-09-16
Changed in cinder:
assignee: nobody → zhangchao (zhangchao010)
status: New → In Progress

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) on 2013-10-04
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-rc1 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers