Dan Kenigsberg has submitted this change and it was merged. Change subject: storageServer: Cleanup logging when mount fails ......................................................................
storageServer: Cleanup logging when mount fails - Log the OSError text if os.rmdir fails instead of an unhelpful traceback - Use logging.exception instead of exc_info=True - Use "mountpoint" instead of "mount point" - Use %r for revealing whitespace bugs in paths - Unify logging messages Change-Id: I24ae2c6ce71a1f20459977a8860dd6a1ebc45db3 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/36371 Reviewed-by: Allon Mureinik <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/storage/storageServer.py 1 file changed, 4 insertions(+), 5 deletions(-) Approvals: Nir Soffer: Verified Allon Mureinik: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24ae2c6ce71a1f20459977a8860dd6a1ebc45db3 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
