Dan Kenigsberg has submitted this change and it was merged. Change subject: Uniform LocalDirectoryConnection.checkTarget() errors. ......................................................................
Uniform LocalDirectoryConnection.checkTarget() errors. Hard to believe: there are uglier things the the True or Exception pattern. Change-Id: Icb8d2afd26efbd7ab228ccf20658485f776802de Signed-off-by: Eduardo <[email protected]> --- M vdsm/storage/storageServer.py M vdsm/storage/storage_exception.py 2 files changed, 13 insertions(+), 6 deletions(-) Approvals: Eduardo: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10150 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb8d2afd26efbd7ab228ccf20658485f776802de Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Eduardo <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Paikov <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
