Skip to content

Commit db36279

Browse files
Failed remote run artifact store (#289)
* Fast-Exit on remote failure. * Fast-Exit on remote failure.
1 parent 8374efd commit db36279

File tree

3 files changed

+183
-164
lines changed

3 files changed

+183
-164
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[tool.poetry]
22
name = "redisbench-admin"
3-
version = "0.6.21"
3+
version = "0.6.22"
44
description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
55
authors = ["filipecosta90 <[email protected]>","Redis Performance Group <[email protected]>"]
66
readme = "README.md"

redisbench_admin/run_remote/remote_client.py

Lines changed: 21 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -102,22 +102,27 @@ def run_remote_client_tool(
102102
benchmark_duration_seconds = calculate_client_tool_duration_and_check(
103103
benchmark_end_time, benchmark_start_time, step_name, warn_min_duration
104104
)
105-
(
106-
artifact_version,
107-
local_bench_fname,
108-
results_dict,
109-
return_code,
110-
) = post_process_remote_run(
111-
artifact_version,
112-
benchmark_config,
113-
benchmark_tool,
114-
local_bench_fname,
115-
return_code,
116-
start_time_ms,
117-
start_time_str,
118-
stdout,
119-
tmp,
120-
)
105+
artifact_version = None
106+
local_bench_fname = None
107+
results_dict = None
108+
return_code = None
109+
if remote_run_result is True:
110+
(
111+
artifact_version,
112+
local_bench_fname,
113+
results_dict,
114+
return_code,
115+
) = post_process_remote_run(
116+
artifact_version,
117+
benchmark_config,
118+
benchmark_tool,
119+
local_bench_fname,
120+
return_code,
121+
start_time_ms,
122+
start_time_str,
123+
stdout,
124+
tmp,
125+
)
121126
return (
122127
artifact_version,
123128
benchmark_duration_seconds,

0 commit comments

Comments
 (0)