From d9460a5e0f0a7aeccc7f4121ce07056084a40912 Mon Sep 17 00:00:00 2001 From: Robin Cernin Date: Wed, 29 Sep 2021 15:14:28 +1000 Subject: [PATCH] Log the exception-info when an error raises in hanlder on the line #114 we set the liveness probe to False, which triggers restart of the Kuryr Controller. However without DEBUG, the LOG message is hidden behind DEBUG. This makes it harder to troubleshoot if the Handler failed with this exception Change-Id: I71deb8434273ce39453d69a05d6bd8b105fe2c2b --- kuryr_kubernetes/handlers/retry.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/kuryr_kubernetes/handlers/retry.py b/kuryr_kubernetes/handlers/retry.py index 1116137ad..772af1fe0 100644 --- a/kuryr_kubernetes/handlers/retry.py +++ b/kuryr_kubernetes/handlers/retry.py @@ -95,7 +95,8 @@ class Retry(base.EventHandler): if self._sleep(deadline, attempt, ex.value): ex.reraise = False else: - LOG.debug('Report handler unhealthy %s', self._handler) + LOG.exception('Report handler unhealthy %s', + self._handler) self._handler.set_liveness(alive=False, exc=ex.value) except Exception as ex: LOG.exception('Report handler unhealthy %s', self._handler)