Francesco Romani has submitted this change and it was merged. Change subject: lvm: Improve logging lv activation logging ......................................................................
lvm: Improve logging lv activation logging Log vg name and lv names when we activate, refresh or deactivate lvs. All are important state changes that should be logged in INFO level. The actual lvm commands are logged in DEBUG level, and may not be available when we change the default log level to INFO. Change-Id: I368b158c8d60ae2ff252e038b01024e4598b5f4d Bug-Url: https://bugzilla.redhat.com/1358348 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: https://gerrit.ovirt.org/61286 Reviewed-by: Allon Mureinik <[email protected]> Reviewed-by: Adam Litke <[email protected]> Continuous-Integration: Jenkins CI Reviewed-on: https://gerrit.ovirt.org/62732 Reviewed-by: Freddy Rolland <[email protected]> Tested-by: Allon Mureinik <[email protected]> Reviewed-by: Francesco Romani <[email protected]> --- M vdsm/storage/lvm.py 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Verified Francesco Romani: Looks good to me, approved Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/62732 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I368b158c8d60ae2ff252e038b01024e4598b5f4d Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
