]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add retry to lvm snapshot create
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 22 Jan 2015 18:22:25 +0000 (11:22 -0700)
committerJohn Griffith <john.griffith8@gmail.com>
Sat, 24 Jan 2015 23:05:31 +0000 (23:05 +0000)
commite302cefafc18754a4495ad061062732b97d326ea
tree94a639a03d6fc212886490e8ce1c00fd69f9374a
parenta8499b24a99f722af94e05ea63f5b75fcf9ba71b
Add retry to lvm snapshot create

We have some occasional issues with snapshot-create
failing for what looks to be conflicts with udev.  It
looks like this problem is a status conflict between LVM
cache and udev, and in most cases the best way to get
around this is to retry the command a few times until
the cache and udev are back in sync.

This patch uses the newly added retry decorator and
for now we're just adding it to the snapshot create
call.  We're using the default values for interval and
retry count but we can certainly adjust this as needed.

Change-Id: Ic884b829591361da979fde968f5710237b34565f
Closes-Bug: #1335905
cinder/brick/local_dev/lvm.py