[add] Fixed spinning/shutting down of local redis. Made the module check use internal addresses (#101)

Author: filipecosta90

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redisbench-admin"
-version = "0.1.74"
+version = "0.1.75"
 description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
 authors = ["filipecosta90 <[email protected]>"]
 readme = "README.md"

redisbench_admin/run_local/run_local.py

@@ -155,11 +155,11 @@ def run_local_command_logic(args):
                 "Some unexpected exception was caught during remote work. Failing test...."
             )
             logging.critical(sys.exc_info())
-    # tear-down
-    logging.info("Tearing down setup")
-    if redis_process is not None:
-        redis_process.kill()
-    logging.info("Tear-down completed")
+        # tear-down
+        logging.info("Tearing down setup")
+        if redis_process is not None:
+            redis_process.kill()
+        logging.info("Tear-down completed")
 
     exit(return_code)
 

redisbench_admin/run_remote/run_remote.py

@@ -88,9 +88,7 @@ def extract_artifact_version_remote(
     server_public_ip, server_public_port, username, private_key
 ):
     commands = [
-        "redis-cli -h {} -p {} info modules".format(
-            server_public_ip, server_public_port
-        ),
+        "redis-cli -h {} -p {} info modules".format("localhost", server_public_port),
     ]
     res = execute_remote_commands(server_public_ip, username, private_key, commands)
     recv_exit_status, stdout, stderr = res[0]