Skip to content

Commit 8433ee3

Browse files
committed
sq-poller: don´t retry and show tracelog when dev discovery raise exc
Signed-off-by: Claudio Lorina <[email protected]>
1 parent 80c75d9 commit 8433ee3

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

suzieq/poller/worker/nodes/node.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -463,6 +463,15 @@ async def _detect_node_type(self):
463463
# that sets the device type
464464
devtype = self.devtype
465465
except Exception:
466+
self.logger.exception(f'{self.address}:{self.port}: Node '
467+
'discovery failed due to exception')
468+
# All the exceptions related to timeouts and authentication
469+
# problems are already catched inside. If we get an
470+
# exception here, this is unexpected and most likely something
471+
# went wrong with the command output parsing.
472+
# In this case there is not point in retrying discovery, it is
473+
# likely a bug.
474+
self._retry = 0
466475
devtype = None
467476

468477
if not devtype:

0 commit comments

Comments
 (0)