Change in vdsm[master]: sp: refreshDomainLinks before convertDomain - vdsm-patches - Fedora mailing-lists

5 Apr 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@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@redhat.com

    