From 0cceabde1f0b6ffb35df61001cf1238ee0f73cfa Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Mon, 3 Oct 2022 14:32:55 -0700 Subject: [PATCH] Add extra logging around tenant reconfiguration events While tracking down a problem, I observed a tenant reconfiguration event apparently go missing. This change adds log messages that might aid debugging in such a situation. Change-Id: Ia975f83b6bba8d48994cd8c39427be2db3d73262 --- zuul/scheduler.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/zuul/scheduler.py b/zuul/scheduler.py index 9548cb6d20..7dd869fae8 100644 --- a/zuul/scheduler.py +++ b/zuul/scheduler.py @@ -1453,6 +1453,7 @@ class Scheduler(threading.Thread): # This is called in the scheduler loop after another thread submits # a request if self.unparsed_abide.ltime < self.system_config_cache.ltime: + self.log.debug("Updating system config") self.updateSystemConfig() with self.layout_lock: @@ -2394,11 +2395,14 @@ class Scheduler(threading.Thread): event_forwarded = False try: if isinstance(event, TenantReconfigureEvent): + self.log.debug("Processing tenant reconfiguration " + "event for tenant %s", tenant.name) self._doTenantReconfigureEvent(event) elif isinstance(event, (PromoteEvent, ChangeManagementEvent)): event_forwarded = self._forward_management_event(event) else: - self.log.error("Unable to handle event %s", event) + self.log.error("Unable to handle event %s for tenant %s", + event, tenant.name) finally: if event_forwarded: self.management_events[tenant.name].ackWithoutResult(