]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Check for silent failure of tgtadm remove
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 10 Apr 2014 00:10:08 +0000 (00:10 +0000)
committerjohn-griffith <john.griffith@solidfire.com>
Mon, 14 Apr 2014 19:24:14 +0000 (13:24 -0600)
commite5c45bf88448b231a3252aa2628d6cf5bd9aa730
treeaf6fd335bd8d1a3fe687699fd5b09f7a95835dcb
parent512170afa377752dce02db3b2c0452d27c8ffd48
Check for silent failure of tgtadm remove

In order to work around a failure to remove targets that
had a reconnect the force flag was added to the tgt delete cmd.
    (https://bugs.launchpad.net/cinder/+bug/1159948)

It turns out there's a bug in tgt where some versions will
sometimes silently fail when using the force flag.
    (https://bugs.launchpad.net/ubuntu/+source/tgt/+bug/1305343)

The problem is that in the gates since we merged the force change
there's a very high number of cases where lvremove fails because
the device is still active.  It appears that this is a result of
the silent force failures in target.

This patch adds a simple check after the force target removal,
if it detects the target is still present it reverts back to the old
non-force method to catch the cases that don't actually need the
force option.

This is a work-around until the version of target in the distros
is updated and can be used reliably, but closes the gate issue
bug that was reported.

Change-Id: I9150669040815e4831bd570964d12676b83ecbc9
Close-Bug: #1304122
(cherry picked from commit f9519182f4a6e6573513d8dbc9438702fc7b8644)
cinder/brick/iscsi/iscsi.py
cinder/tests/test_iscsi.py