Igor Lvovsky has posted comments on this change. Change subject: [WIP]Extracted image related functinality to another class ......................................................................
Patch Set 8: (1 inline comment) .................................................... File vdsm/storage/sp.py Line 198: self.spmRole = SPM_FREE Line 199: self._imageManipulator = LegacyImageManipulator(os.path.join(self.storage_repository, self.spUUID)) Line 200: Line 201: # I know it exposes everything as unsecured. Line 202: # I DON'T CARE. Will be handeled later. where? when? Line 203: @unsecured Line 204: def getImageManipulator(self): Line 205: return self._imageManipulator Line 206: -- To view, visit http://gerrit.usersys.redhat.com/1074 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic29dc8d964f0c2e56329e7044a3b2f48a9f2e193 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
