From 97c8656b08f1e9a07ad89ca2b19a76e34f2ee431 Mon Sep 17 00:00:00 2001 From: Trevor Benson Date: Fri, 29 Dec 2023 11:01:28 -0800 Subject: [PATCH] [transport] Prevent package list for unresponsive nodes Raise error when _connect() returns non zero status. Signed-off-by: Trevor Benson --- sos/collector/transports/saltstack.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sos/collector/transports/saltstack.py b/sos/collector/transports/saltstack.py index 3691a36392..f0660c8aa1 100644 --- a/sos/collector/transports/saltstack.py +++ b/sos/collector/transports/saltstack.py @@ -106,6 +106,8 @@ def _connect(self, password=None): self.log_info("Transport is locally supported and service running. ") cmd = "echo Connected" result = self.run_command(cmd, timeout=180) + if result['status'] == 1: + raise ConnectionException(self.address) return result['status'] == 0 def _disconnect(self):