Dan Kenigsberg has submitted this change and it was merged. Change subject: readlines - remove redundant pathExists check ......................................................................
readlines - remove redundant pathExists check fileSD -> readlines: * The verification of path existence is redundant. * Try to read the metadata file and handle an exception: - If no such file: return an empty list. - Else, raise the exception. Change-Id: Id73f4bd177465f253eb4f65403b17ff7375e329c Bug-Url: https://bugzilla.redhat.com/882667 Signed-off-by: Daniel Erez <[email protected]> --- M vdsm/storage/fileSD.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Daniel Erez: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13664 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id73f4bd177465f253eb4f65403b17ff7375e329c Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Ohad Basan <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
