[fix] Shutting down remote redis properly (#174)

Author: filipecosta90

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redisbench-admin"
-version = "0.4.1"
+version = "0.4.2"
 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/cli.py

@@ -27,7 +27,7 @@
 
 
 LOG_LEVEL = logging.DEBUG
-if os.getenv("VERBOSE", "1") == "0":
+if os.getenv("VERBOSE", "0") == "0":
     LOG_LEVEL = logging.INFO
 LOG_FORMAT = "%(asctime)s %(levelname)-4s %(message)s"
 LOG_DATEFMT = "%Y-%m-%d %H:%M:%S"

redisbench_admin/run_remote/run_remote.py

@@ -321,8 +321,6 @@ def run_remote_command_logic(args, project_name, project_version):
                                 benchmark_config, local_redis_conn, required_modules
                             )
 
-                            ssh_tunnel.close()  # Close the tunnel
-
                             (
                                 benchmark_min_tool_version,
                                 benchmark_min_tool_version_major,
@@ -392,6 +390,10 @@ def run_remote_command_logic(args, project_name, project_version):
                                 command_str,
                             )
                             benchmark_end_time = datetime.datetime.now()
+
+                            logging.info("Shutting down remote redis.")
+                            local_redis_conn.shutdown(save=False)
+                            ssh_tunnel.close()  # Close the tunnel
                             if remote_run_result is False:
                                 failed_remote_run_artifact_store(
                                     args,