openstack
/
cinder
Code Issues Proposed changes

Merge "Restore_backup: set error when volume status not matched"

This commit is contained in:
Zuul 2017-12-14 00:59:37 +00:00 committed by Gerrit Code Review
parent c30538e95d 06d4ba2080
commit 526c1a7372
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/backup/manager.py
View File

@ -466,6 +466,8 @@ class BackupManager(manager.ThreadPoolManager):
'actual_status': actual_status})
backup.status = fields.BackupStatus.AVAILABLE
backup.save()
self.db.volume_update(context, volume_id,
{'status': 'error_restoring'})
raise exception.InvalidVolume(reason=err)
expected_status = fields.BackupStatus.RESTORING

2
cinder/tests/unit/backup/test_backup.py
View File

@ -940,6 +940,8 @@ class BackupTestCase(BaseBackupTest):
backup,
vol_id)
backup = db.backup_get(self.ctxt, backup.id)
vol = db.volume_get(self.ctxt, vol_id)
self.assertEqual('error_restoring', vol['status'])
self.assertEqual(fields.BackupStatus.AVAILABLE, backup['status'])
def test_restore_backup_with_bad_backup_status(self):