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