Allon Mureinik has uploaded a new change for review.

Change subject: WIP: sd.py: Remove sds from sdCache when connecting
......................................................................

WIP: sd.py: Remove sds from sdCache when connecting

When connecting to a storage server, pre-existing domains should be
removed, since we have no way of knowing what happened to these
domains in the meanwhile.

Change-Id: Ic0e0d0e970ce55acf92f7e39ec9cf2170e948274
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=950055
Signed-off-by: Allon Mureinik <[email protected]>
---
M vdsm/storage/hsm.py
1 file changed, 5 insertions(+), 0 deletions(-)


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

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index d46ef50..8fae844 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -2371,6 +2371,11 @@
                     self.log.debug("prefetch failed: %s",
                                    sdCache.knownSDs,  exc_info=True)
                 else:
+                    # Any pre-existing domains in sdCache stand the chance of
+                    # being invalid, since there is no way to know what happens
+                    # to them while the storage is disconnected.
+                    for sdUUID in doms.iterkeys():
+                        sdCache.manuallyRemoveDomain(sdUUID)
                     sdCache.knownSDs.update(doms)
 
             self.log.debug("knownSDs: {%s}", ", ".join("%s: %s.%s" %


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic0e0d0e970ce55acf92f7e39ec9cf2170e948274
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to