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