openstack
/
cinder
Code Issues Proposed changes

Merge "Remove unnecessary saving of host and availability_zone"

This commit is contained in:
Zuul 2019-11-27 18:43:41 +00:00 committed by Gerrit Code Review
parent 2c9310e243 b5b6443adc
commit 4d2e71e13c
1 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
cinder/backup/manager.py
View File

@ -358,10 +358,6 @@ class BackupManager(manager.ThreadPoolManager):
self._notify_about_backup_usage(context, backup, "create.start")
backup.host = self.host
backup.availability_zone = self.az
backup.save()
expected_status = "backing-up"
if snapshot_id:
actual_status = snapshot['status']
@ -536,9 +532,6 @@ class BackupManager(manager.ThreadPoolManager):
volume = objects.Volume.get_by_id(context, volume_id)
self._notify_about_backup_usage(context, backup, "restore.start")
backup.host = self.host
backup.save()
expected_status = [fields.VolumeStatus.RESTORING_BACKUP,
fields.VolumeStatus.CREATING]
volume_previous_status = volume['status']
@ -715,8 +708,6 @@ class BackupManager(manager.ThreadPoolManager):
LOG.info('Delete backup started, backup: %s.', backup.id)
self._notify_about_backup_usage(context, backup, "delete.start")
backup.host = self.host
backup.save()
expected_status = fields.BackupStatus.DELETING
actual_status = backup.status