Change in vdsm[master]: sp: refreshDomainLinks before convertDomain

fsimonce at redhat.com fsimonce at redhat.com
Fri Apr 5 12:35:20 UTC 2013


Federico Simoncelli has uploaded a new change for review.

Change subject: sp: refreshDomainLinks before convertDomain
......................................................................

sp: refreshDomainLinks before convertDomain

Domain conversion requires the links to be present.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=948346
Change-Id: If5d2481235bb54119be864a4a6bbb220718ade55
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/storage/sp.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/33/13633/1

diff --git a/vdsm/storage/sp.py b/vdsm/storage/sp.py
index 82052df..fe54dd9 100644
--- a/vdsm/storage/sp.py
+++ b/vdsm/storage/sp.py
@@ -1109,6 +1109,8 @@
         if domainStatuses[sdUUID] == sd.DOM_ACTIVE_STATUS:
             return True
 
+        # Domain conversion requires the links to be present
+        self._refreshDomainLinks(dom)
         if dom.getDomainClass() == sd.DATA_DOMAIN:
             self._convertDomain(dom)
 
@@ -1116,7 +1118,6 @@
         # set domains also do rebuild
         domainStatuses[sdUUID] = sd.DOM_ACTIVE_STATUS
         self.setMetaParam(PMDK_DOMAINS, domainStatuses)
-        self._refreshDomainLinks(dom)
         self.updateMonitoringThreads()
         return True
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If5d2481235bb54119be864a4a6bbb220718ade55
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>


More information about the vdsm-patches mailing list