From 1ef74537fbf652c38088c8706c0cd8bbd60efb40 Mon Sep 17 00:00:00 2001 From: Sundaresan Rajangam Date: Wed, 11 May 2016 14:50:58 -0700 Subject: [PATCH] Don't log traceback for redis ConnectionError exception In partition_handler, the redis connection timesout if there is no activity for 90 seconds. We have catch all exception that logs the traceback. Need to catch the ConnectionError exception and not log the traceback for this case. Change-Id: If2f391d5ddf3a04aed82b889b1d780de60c4da6b Closes-Bug: #1578779 (cherry picked from commit 2539e2e1ed9bab1cffce2fa47ebc066d3693c53d) --- src/opserver/partition_handler.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/opserver/partition_handler.py b/src/opserver/partition_handler.py index b72ba8dc3e5..f9f1373a632 100644 --- a/src/opserver/partition_handler.py +++ b/src/opserver/partition_handler.py @@ -417,11 +417,14 @@ def _run(self): except gevent.GreenletExit: break + except redis.exceptions.ConnectionError: + pass except Exception as ex: template = "Exception {0} in uve stream proc. Arguments:\n{1!r}" messag = template.format(type(ex).__name__, ex.args) self._logger.error("%s : traceback %s" % \ (messag, traceback.format_exc())) + finally: lredis = None if pb is not None: pb.close()