From a6fa7b80238699bb254619b33dd7286152690057 Mon Sep 17 00:00:00 2001 From: Xavier-Do Date: Thu, 9 Jun 2022 08:35:00 +0200 Subject: [PATCH] [FIX] runbot: fix leader pull_info_failures The initial pull_info_failures feature was introduced before the separate process for the leader. Since this is here for a rare case, this wasn't tested since then. Testing the new infrastructure without token, there was many pull_info_failure, leading to an error on cleanup. --- runbot/models/runbot.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/runbot/models/runbot.py b/runbot/models/runbot.py index 23245784..6e5792a5 100644 --- a/runbot/models/runbot.py +++ b/runbot/models/runbot.py @@ -262,7 +262,7 @@ class Runbot(models.AbstractModel): for pr_number, t in pull_info_failures.items(): if t + 15*60 < time.time(): _logger.warning('Removing %s from pull_info_failures', pr_number) - del self.pull_info_failures[pr_number] + del pull_info_failures[pr_number] return manager.get('sleep', default_sleep)