Change in vdsm[master]: Add logging to rules operations in supervdsmServer
danken at redhat.com
danken at redhat.com
Mon Mar 31 16:29:02 UTC 2014
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Add logging to rules operations in supervdsmServer
......................................................................
Add logging to rules operations in supervdsmServer
Previously when udev rules were created and removed, we had no logging,
so trying to debug issues with udev rules was very hard. This patch log
creation and removal of udev rules.
Although the whole methods were logged using logDecorator the creation
and removal of each rule was not logged independently
Change-Id: Ib1d8ede0cbe0790bdfcd60ed833b3d9c54f31eba
Signed-off-by: Xavi Francisco <xfrancis at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26121
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Xavi Francisco: Verified
Douglas Schilling Landgraf: Looks good to me, but someone else must approve
Nir Soffer: Looks good to me, but someone else must approve
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/26121
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1d8ede0cbe0790bdfcd60ed833b3d9c54f31eba
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xavi Francisco <xfrancis at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Xavi Francisco <xfrancis at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list