Dan Kenigsberg has submitted this change and it was merged. Change subject: domainMonitor: Cleanup logging ......................................................................
domainMonitor: Cleanup logging - Use INFO level for the important state change message, and uppercase VALID and INVALID for extra visibility. - Add few debug messages that may help debugging. - Replace non-standard exc_info with logging.exception. - Simplify messages text and remove unneeded words. Change-Id: I7c24ad9f9c2c4f9b57e0682f4ab3325617b46dd9 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/32505 Reviewed-by: Federico Simoncelli <[email protected]> --- M vdsm/storage/domainMonitor.py 1 file changed, 16 insertions(+), 17 deletions(-) Approvals: Nir Soffer: Verified Federico Simoncelli: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32505 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c24ad9f9c2c4f9b57e0682f4ab3325617b46dd9 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[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: Yoav Kleinberger <[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
