]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: stop running "true" before every command 59610/head
authorAdam King <adking@redhat.com>
Fri, 6 Sep 2024 13:04:37 +0000 (09:04 -0400)
committerAdam King <adking@redhat.com>
Fri, 6 Sep 2024 13:04:37 +0000 (09:04 -0400)
It turns out this is completely unnecessary. With the keepalive
settings on the connection, we no longer need to run an
additional command with a timeout on it to quickly detect a
host is offline

Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/ssh.py

index 4a30c067709433a5c688cb2b7ac0e95ad4abfd4a..1622cb001abaa1fc93a57291a1175d5495841fbe 100644 (file)
@@ -242,10 +242,6 @@ class SSHManager:
         if log_command:
             logger.debug(f'Running command: {rcmd}')
         try:
-            test_cmd = RemoteSudoCommand(
-                Executables.TRUE, [], use_sudo=use_sudo
-            )
-            r = await conn.run(str(test_cmd), check=True, timeout=5)  # host quick check
             r = await conn.run(str(rcmd), input=stdin)
         # handle these Exceptions otherwise you might get a weird error like
         # TypeError: __init__() missing 1 required positional argument: 'reason' (due to the asyncssh error interacting with raise_if_exception)