openstack/cinder
Code Issues Proposed changes

Race in Cinder backup manager

Browse Source 
Fix for a race in Cinder Backup Manager.

Closes-Bug: 1887859
Change-Id: Ia135b2057118960bb9a395297efaadd0e7ba2720
This commit is contained in:
Alex Deiter 2020-07-16 20:46:00 +00:00
parent 1bb4d40ed5
commit 9893b1c958
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/backup/manager.py
View File

@ -1089,8 +1089,6 @@ class BackupManager(manager.SchedulerDependentManager):
self._publish_service_capabilities(context) self._publish_service_capabilities(context)
def _report_driver_status(self, context): def _report_driver_status(self, context):
if not self.is_working():
self.setup_backup_backend(context)
backup_stats = { backup_stats = {
'backend_state': self.is_working(), 'backend_state': self.is_working(),
'driver_name': self.driver_name, 'driver_name': self.driver_name,

6
releasenotes/notes/bug-1887859-backup-manager-fb8dbf289eedc4b0.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
`Bug #1887859 <https://bugs.launchpad.net/cinder/+bug/1887859>`_:
Fix for a race in Cinder Backup Manager with double initialization
of backup driver.