From 3b6fdc147e01e630b836d7af917d7f17b11b986d Mon Sep 17 00:00:00 2001 From: Ross McFarland Date: Tue, 14 Oct 2025 11:19:13 -0700 Subject: [PATCH] Fix logging issues in Manager: convert generator to list and correct log method prefix --- .changelog/bec5cc4923da4128b90cd6bc6d66af76.md | 4 ++++ octodns/manager.py | 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) create mode 100644 .changelog/bec5cc4923da4128b90cd6bc6d66af76.md diff --git a/.changelog/bec5cc4923da4128b90cd6bc6d66af76.md b/.changelog/bec5cc4923da4128b90cd6bc6d66af76.md new file mode 100644 index 0000000..86e13bf --- /dev/null +++ b/.changelog/bec5cc4923da4128b90cd6bc6d66af76.md @@ -0,0 +1,4 @@ +--- +type: none +--- +Fix logging issues in Manager: convert generator to list and correct log method prefix diff --git a/octodns/manager.py b/octodns/manager.py index a7894d5..c14c7ea 100644 --- a/octodns/manager.py +++ b/octodns/manager.py @@ -570,7 +570,7 @@ class Manager(object): ]: return None - self.log.info('sync: sources=%s', sources) + self.log.info('_get_sources: sources=%s', sources) try: # rather than using a list comprehension, we break this loop @@ -630,7 +630,7 @@ class Manager(object): self.log.info( '_preprocess_zones: dynamic zone=%s, sources=%s', name, - (s.id for s in found_sources), + list(s.id for s in found_sources), ) candidates = set() for source in found_sources: