]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Improved fail_reason for cinder-backup swift connection errors
authorStephen Mulcahy <stephen.mulcahy@hp.com>
Thu, 28 Feb 2013 12:11:39 +0000 (12:11 +0000)
committerStephen Mulcahy <stephen.mulcahy@hp.com>
Thu, 28 Feb 2013 13:16:23 +0000 (13:16 +0000)
commit3fc515e77aa8a6bc3218dbd36b8ffbb9e7bf5b97
tree6a31e1f06c44effc789b0e755852487a118407e8
parentdd010690330e8304eadddab158ba5c2e3f57d8c8
Improved fail_reason for cinder-backup swift connection errors

Modified swift backup service to catch socket errors when talking to
swift and raise a specific SwiftConnectionFailed exception in these
cases. This allows us to provide a more readable error message
detailing the problem connecting to swift to the end user when they
view the backup. Also reduced the default number of swift retries
so devstack environments fail faster - production environments can
tune these in cinder.conf.

Fixes bug: 1132791

Change-Id: Ibca744ea5adcbd31d068ac3d858bde6a4a0c9844
cinder/backup/services/swift.py
cinder/exception.py
cinder/tests/backup/fake_swift_client.py
cinder/tests/test_backup_swift.py