]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 26f1e8e) | patch
Don't fail target_delete if ACL's don't exist
authorMike Perez <thingee@gmail.com>
Sat, 7 Mar 2015 01:24:00 +0000 (17:24 -0800)
committerMike Perez <thingee@gmail.com>
Sat, 7 Mar 2015 01:28:54 +0000 (17:28 -0800)
commit9f49be03b2e53b9e5eae36b58ba03ee3ff3e4ff4
tree63091b7b5cd492092a0007575527353d5192f80btree | snapshot
parent26f1e8e296c8b980069c15f015728817bc0c76fccommit | diff
Don't fail target_delete if ACL's don't exist

Seen in the gate a bit where a target delete is called and fails because
the ACL's don't exist for that target. If we're deleting a target and
the ACL's don't exist, it's probably safe to move on.

This patch adds a check for this specific case, logs a warning, and
continues rather than raising an exception.

Closes-Bug: #1425310
Change-Id: Ideada8ddea8624aff7add95e9caad2aba4e31ae3
cinder/tests/targets/test_tgt_driver.py diff | blob | history
cinder/volume/targets/tgt.py diff | blob | history