Skip to content
Snippets Groups Projects
Commit ce955fba authored by Nicolas Dandrimont's avatar Nicolas Dandrimont
Browse files

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.
parent 61cfd77d
No related branches found
No related tags found
No related merge requests found
Pipeline #14125 passed
......@@ -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)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment