Storwize cinder volume driver have unused error message.
Bug #1266321 reported by
Jia Ming Zhang
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Low
|
Jia Ming Zhang |
Bug Description
In the cinder volume Storwize driver, the do_setup will check the iogrp is ready or not. If the io_grp data read have exception raised, there's a legacy 'vdisk_count' which is legacy and unnecessary for this ValueError.
The 'vdisk': iogrp_data[
Changed in cinder: | |
assignee: | nobody → Jia Ming Zhang (jmzhang) |
Changed in cinder: | |
status: | New → Confirmed |
status: | Confirmed → In Progress |
Changed in cinder: | |
importance: | Undecided → Low |
Changed in cinder: | |
milestone: | none → icehouse-2 |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
milestone: | icehouse-2 → 2014.1 |
To post a comment you must log in.
The code snippet is as following. The iogrp_data[ 'vdisk_ count'] should be a legacy variable to be checked but it is no-longer used now.
except ValueError:
'svcinfo lsiogrp returned: %(node)s and %(vdisk)s') %
{'node' : iogrp_data[ 'node_count' ],
'vdisk' : iogrp_data[ 'vdisk_ count'] })
msg = (_('Expected integers for node_count and vdisk_count, '