From 5f713504e5a53af86bbf4e783a7631ef295366ac Mon Sep 17 00:00:00 2001
From: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Wed, 19 Mar 2025 11:15:30 +0100
Subject: [PATCH] pattern: only log disabling origins once
Getting this message on every page is needlessly verbose, especially in
staging where most listers are configured this way.
---
swh/lister/pattern.py | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/swh/lister/pattern.py b/swh/lister/pattern.py
index ec697e80..fd930a89 100644
--- a/swh/lister/pattern.py
+++ b/swh/lister/pattern.py
@@ -218,6 +218,7 @@ class Lister(Generic[StateType, PageType]):
"""
full_stats = ListerStats()
self.recorded_origins = set()
+ disabling_logged = False
try:
for page in self.get_pages():
@@ -237,9 +238,11 @@ class Lister(Generic[StateType, PageType]):
break
if not self.enable_origins:
- logger.info(
- "Disabling origins before sending them to the scheduler"
- )
+ if not disabling_logged:
+ logger.info(
+ "Disabling origins before sending them to the scheduler"
+ )
+ disabling_logged = True
origins = [attr.evolve(origin, enabled=False) for origin in origins]
sent_origins = self.send_origins(origins)
self.recorded_origins.update(sent_origins)
--
GitLab