Ayal Baron has posted comments on this change. Change subject: sdcache: add refresh to connectStoragePool ......................................................................
Patch Set 1: (2 inline comments) .................................................... File vdsm/storage/sp.py Line 638: self._saveReconnectInformation(hostID, scsiKey, msdUUID, masterVersion) Line 639: self.id = hostID Line 640: self.scsiKey = scsiKey Line 641: # Make sure SDCache doesn't have stale data (it can be in case of FC) Line 642: sdCache.invalidateStorage() why is this not needed in refresh storage pool? (i.e. sp.py - refresh method) which is the only other method which calls sdCache.refresh(0 Line 643: sdCache.refresh() Line 644: # Rebuild whole Pool Line 645: self.__rebuild(msdUUID=msdUUID, masterVersion=masterVersion) Line 646: self.__createMailboxMonitor() Line 639: self.id = hostID Line 640: self.scsiKey = scsiKey Line 641: # Make sure SDCache doesn't have stale data (it can be in case of FC) Line 642: sdCache.invalidateStorage() Line 643: sdCache.refresh() why isn't this needed in connectStorageServer? (the only other place which calls invalidateStorage)? Line 644: # Rebuild whole Pool Line 645: self.__rebuild(msdUUID=msdUUID, masterVersion=masterVersion) Line 646: self.__createMailboxMonitor() Line 647: -- To view, visit http://gerrit.ovirt.org/9422 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2d3adcff7bb0e97be5c797cd720c6353920d9db0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
