]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Brick LVM: LV not found logging and error handling
authorEric Harney <eharney@redhat.com>
Fri, 21 Nov 2014 16:50:48 +0000 (11:50 -0500)
committerEric Harney <eharney@redhat.com>
Tue, 25 Nov 2014 13:57:29 +0000 (08:57 -0500)
commit8a08a6cdd3688e799ae2db4bd94b8b14dc50f2a9
tree6e58ac02c3cff36c6e37c6c9d9f8c0cbb7c195b2
parent3ec99175818d01eaad09c7947c862593846f3b51
Brick LVM: LV not found logging and error handling

Fix up two things in get_lv_info/get_volume:
1. ProcessExecutionError should be handled where we call the
   command.  We can just return an empty list for this case
   which makes things simple for callers and consistent with
   querying a VG.
2. Not found errors should be logged as info and not warning
   since this is generally not actionable by the admin (and not
   a problem).

Fix typo in lvm command output for not found test.

Add test for get_lv_info not found error.

Change-Id: Iebccf7b8f252303f586b36aad33b85945ea5c927
Related-Bug: #1390081
cinder/brick/local_dev/lvm.py
cinder/tests/brick/test_brick_lvm.py