]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Changed pvs separator from ':' to '|' to support names with ':'.
authorJoe D'Andrea <jdandrea@research.att.com>
Fri, 16 Jan 2015 21:26:09 +0000 (21:26 +0000)
committerJoe D'Andrea <jdandrea@research.att.com>
Mon, 26 Jan 2015 18:35:00 +0000 (18:35 +0000)
commit456a6f81d8da03ef852575e1e6053dae6958dd43
tree06f0d73b4c89b57c35e2cc6e8ae186449d6c7f4a
parent4e11ef7343ee86e6eca1380d9d8495b641183597
Changed pvs separator from ':' to '|' to support names with ':'.

get_all_physical_volumes() runs:

pvs --noheadings --unit=g -o vg_name,name,size,free
    --separator : --nosuffix

Which can produce a variable number of fields if the 'name'
incorporates ':', as in the following:

cinder-volumes-array-lun-0:/dev/disk/by-path/ip-10.8.77.11:3260-
    iscsi-iqn.1984-05.com.dell:powervault.md3600i.6f01faf000eb1b
    540000000053c487b0-lun-0:931.01:931.01

Change-Id: Ia5442af2a06a004ff1af58ce08b816506b369f22
Closes-Bug: 1414076
cinder/brick/local_dev/lvm.py
cinder/tests/brick/test_brick_lvm.py
cinder/tests/test_srb.py