Change in vdsm[master]: Add logging to fs operations in supervdsmServer

xfrancis at redhat.com xfrancis at redhat.com
Wed Mar 26 15:07:28 UTC 2014


Xavi Francisco has uploaded a new change for review.

Change subject: Add logging to fs operations in supervdsmServer
......................................................................

Add logging to fs operations in supervdsmServer

The motivation of this patch is to increase the logging of some
filesystem related operations in supervdsmServer. Although most of
the operations are logged through the logDecorator in some cases
this does not provide enough granularity to detect problems.

Change-Id: Ib1d8ede0cbe0790bdfcd60ed833b3d9c54f31eba
Signed-off-by: Xavi Francisco <xfrancis at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/21/26121/1

diff --git a/vdsm/supervdsmServer b/vdsm/supervdsmServer
index 13c14f9..d284a43 100755
--- a/vdsm/supervdsmServer
+++ b/vdsm/supervdsmServer
@@ -280,6 +280,7 @@
         for r in rules:
             try:
                 os.remove(r)
+                self.log.debug("Rule %s removed", r)
             except OSError:
                 fails.append(r)
         return fails
@@ -396,6 +397,7 @@
         log.debug("Cleaning old socket %s", address)
         if os.path.exists(address):
             os.unlink(address)
+            log.debug("Old socket %s cleaned", address)
 
         log.debug("Setting up keep alive thread")
 
@@ -425,6 +427,7 @@
         finally:
             if os.path.exists(address):
                 utils.rmFile(address)
+                log.debug("Removed socket %s", address)
 
     except Exception:
         log.error("Could not start Super Vdsm", exc_info=True)


-- 
To view, visit http://gerrit.ovirt.org/26121
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib1d8ede0cbe0790bdfcd60ed833b3d9c54f31eba
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xavi Francisco <xfrancis at redhat.com>


More information about the vdsm-patches mailing list