Change in vdsm[master]: sp: setDomainRegularRole is implemented by backend

nsoffer at redhat.com nsoffer at redhat.com
Sat Feb 15 11:28:20 UTC 2014


Nir Soffer has uploaded a new change for review.

Change subject: sp: setDomainRegularRole is implemented by backend
......................................................................

sp: setDomainRegularRole is implemented by backend

Commit ab6014b394 factored out metadata access to pool backend, but left
one call to StoragePool.setDomainRegularRole() that is implemented now
by the backend. Detected by pylint.

Change-Id: Ia49e45ccab0b933a8f1abc988ea0bd60e7b99b7d
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M vdsm/storage/sp.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/95/24495/1

diff --git a/vdsm/storage/sp.py b/vdsm/storage/sp.py
index a020e1e..162f061 100644
--- a/vdsm/storage/sp.py
+++ b/vdsm/storage/sp.py
@@ -810,7 +810,7 @@
         except Exception:
             self.log.exception('migration to new master failed')
             try:
-                self.setDomainRegularRole(newmsd)
+                self._backend.setDomainRegularRole(newmsd)
             except Exception:
                 self.log.exception('unable to mark domain %s as regular',
                                    newmsd.sdUUID)


-- 
To view, visit http://gerrit.ovirt.org/24495
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia49e45ccab0b933a8f1abc988ea0bd60e7b99b7d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list