]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Set vg_thin_pool to pool name instead of pool_path
authorFlaper Fesp <flaper87@gmail.com>
Tue, 3 Sep 2013 16:26:42 +0000 (18:26 +0200)
committerFlaper Fesp <flaper87@gmail.com>
Tue, 3 Sep 2013 16:33:44 +0000 (18:33 +0200)
commit29e889bde9364f30c3f1cbeb7ee835b036451cba
tree9d700f8b0f5a7bb6d55b2aa0bb6983b8e9b91bd8
parent59df774239ac0783826762108768d79ebb4c0dd5
Set vg_thin_pool to pool name instead of pool_path

create_thin_pool is setting vg_thin_pool to the pool path instead of the
pool_name. This makes volumes creation fail when the create_thin_pool
method is called. This happens because create_volume builds the pool
path itself as create_thin_pool does.

Keeping the pool name in vg_thin_pool instead of the path makes more
sense and allows it to be used in other places in the brick. Also, most
commands return both vg_name and pool_name separated.

Change-Id: Ibf5cd746fc050eab5ce6aff13dd70c1e8066b228
Closes-Bug: #1220286
cinder/brick/local_dev/lvm.py
cinder/tests/brick/test_brick_lvm.py