]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add a retry to create_iscsi_target for LVM
authorJohn Griffith <john.griffith@solidfire.com>
Fri, 20 Sep 2013 02:47:22 +0000 (20:47 -0600)
committerJohn Griffith <john.griffith@solidfire.com>
Fri, 20 Sep 2013 17:09:51 +0000 (11:09 -0600)
commit90f200682c94b1e4b69e04befc045e7ae9111cf7
treed79c35d7d357651b5eccf5a224c635784a3b21e3
parentdee4c862f241a6f12fdb6f0e7c48b32063ef6fe5
Add a retry to create_iscsi_target for LVM

There's a bug in the tgt driver where under certain
conditions where a race condition is present that can
result in tgtadm attempting re-use a target ID.

Testing this it seems that a retry will address this nicely
and cause tgtd to grab a new target ID and things are ok.

This patch adds a simple check/retry mechanism around the
iscsi_target_create in the LVM driver to catch this issue and
deal with it.

There are a number of bugs that it appears can be attributed to
this, but this patch is associated with the most predominant
clear cut version.

Closes-Bug #1223469

Change-Id: I5126009f196adcafad55e73ff99a59262dd93dfe
cinder/volume/drivers/lvm.py