Change in vdsm[master]: [2/x] BindingXMLRPC cleanup: Remove unused variables

amuller at redhat.com amuller at redhat.com
Tue Dec 31 12:18:04 UTC 2013


Assaf Muller has uploaded a new change for review.

Change subject: [2/x] BindingXMLRPC cleanup: Remove unused variables
......................................................................

[2/x] BindingXMLRPC cleanup: Remove unused variables

Change-Id: I1d36a9f71b4d192b8ba825e95055dd42fc278248
Signed-off-by: Assaf Muller <amuller at redhat.com>
---
M vdsm/BindingXMLRPC.py
1 file changed, 0 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/36/22836/1

diff --git a/vdsm/BindingXMLRPC.py b/vdsm/BindingXMLRPC.py
index 4c78bdf..1ffed25 100644
--- a/vdsm/BindingXMLRPC.py
+++ b/vdsm/BindingXMLRPC.py
@@ -18,7 +18,6 @@
 # Refer to the README and COPYING files for full details of the license
 #
 
-import time
 from errno import EINTR
 import SimpleXMLRPCServer
 from vdsm import SecureXMLRPCServer
@@ -103,8 +102,6 @@
             def log_request(self, code='-', size='-'):
                 """Track from where client connections are coming."""
                 self.server.lastClient = self.client_address[0]
-                self.server.lastClientTime = time.time()
-                self.server.lastServerIP = self.request.getsockname()[0]
                 # FIXME: The editNetwork API uses this log file to
                 # determine if this host is still accessible.  We use a
                 # file (rather than an event) because editNetwork is
@@ -143,9 +140,7 @@
                 requestHandler=LoggingHandler, logRequests=True)
         utils.closeOnExec(server.socket.fileno())
 
-        server.lastClientTime = 0
         server.lastClient = '0.0.0.0'
-        server.lastServerIP = '0.0.0.0'
 
         return server
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d36a9f71b4d192b8ba825e95055dd42fc278248
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amuller at redhat.com>


More information about the vdsm-patches mailing list