Change in vdsm[master]: asyncore: remove misleading log entry

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Tue Nov 10 15:10:05 UTC 2015


Piotr Kliczewski has uploaded a new change for review.

Change subject: asyncore: remove misleading log entry
......................................................................

asyncore: remove misleading log entry

>From time to time I can see misleading warning in the logs:

betterAsyncore::144::vds.dispatcher::(log_info) unhandled close event

In order to get rid of it we make sure that we handle close event for
all the dispatchers that we have and we call close on dispatcher.


Change-Id: Ib35eb35c7ffe52b090c329136041bc21694fbdaf
Signed-off-by: pkliczewski <piotr.kliczewski at gmail.com>
---
M lib/vdsm/sslutils.py
M vdsm/protocoldetector.py
2 files changed, 9 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/75/48375/1

diff --git a/lib/vdsm/sslutils.py b/lib/vdsm/sslutils.py
index 443ae8c..3b52a81 100644
--- a/lib/vdsm/sslutils.py
+++ b/lib/vdsm/sslutils.py
@@ -297,6 +297,9 @@
             self.want_read = self.want_write = True
             self._is_handshaking = False
 
+    def handle_close(self, dispatcher):
+        dispatcher.close()
+
 
 def create_ssl_context():
         sslctx = None
diff --git a/vdsm/protocoldetector.py b/vdsm/protocoldetector.py
index 0bcc7a6..af7818d 100644
--- a/vdsm/protocoldetector.py
+++ b/vdsm/protocoldetector.py
@@ -72,6 +72,9 @@
                           *client.getpeername())
             self._dispatcher_factory(client)
 
+    def handle_close(self, dispatcher):
+        dispatcher.close()
+
 
 class _ProtocolDetector(object):
     log = logging.getLogger("ProtocolDetector.Detector")
@@ -128,6 +131,9 @@
     def has_expired(self):
         return monotonic_time() >= self._give_up_at
 
+    def handle_close(self, dispatcher):
+        dispatcher.close()
+
 
 class MultiProtocolAcceptor:
     """


-- 
To view, visit https://gerrit.ovirt.org/48375
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib35eb35c7ffe52b090c329136041bc21694fbdaf
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>


More information about the vdsm-patches mailing list