redis-benchmarks-specification 0.1.88__tar.gz → 0.1.90__tar.gz

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: redis-benchmarks-specification
-Version: 0.1.88
+Version: 0.1.90
 Summary: The Redis benchmarks specification describes the cross-language/tools requirements and expectations to foster performance and observability standards around redis related technologies. Members from both industry and academia, including organizations and individuals are encouraged to contribute.
 Author: filipecosta90
 Author-email: filipecosta.90@gmail.com

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redis-benchmarks-specification"
-version = "0.1.88"
+version = "0.1.90"
 description = "The Redis benchmarks specification describes the cross-language/tools requirements and expectations to foster performance and observability standards around redis related technologies. Members from both industry and academia, including organizations and individuals are encouraged to contribute."
 authors = ["filipecosta90 <filipecosta.90@gmail.com>","Redis Performance Group <performance@redis.com>"]
 readme = "Readme.md"

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/redis_benchmarks_specification/__builder__/builder.py

@@ -506,6 +506,7 @@ def builder_process_stream(
                             tests_priority_lower_limit,
                             tests_priority_upper_limit,
                             tests_regexp,
+                            build_start_datetime,
                             build_duration_secs,
                         )
                         if contains_regression_comment:

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/redis_benchmarks_specification/__cli__/cli.py

@@ -351,6 +351,7 @@ def trigger_tests_cli_command_logic(args, project_name, project_version):
                     )
 
                     if args.wait_build is True:
+                        build_start_datetime = datetime.datetime.utcnow()
                         decoded_stream_id = stream_id.decode()
                         builder_list_streams = (
                             f"builder:{decoded_stream_id}:builds_completed"
@@ -409,6 +410,9 @@ def trigger_tests_cli_command_logic(args, project_name, project_version):
                         logging.info(
                             f"FINAL total of {len_list} already build benchmark stream ids for this build: {benchmark_stream_ids}"
                         )
+                        build_end_datetime = datetime.datetime.utcnow()
+                        build_duration = build_end_datetime - build_start_datetime
+                        build_duration_secs = build_duration.total_seconds()
 
                         comment_body = generate_build_finished_pr_comment(
                             benchmark_stream_ids,
@@ -420,6 +424,8 @@ def trigger_tests_cli_command_logic(args, project_name, project_version):
                             tests_priority_lower_limit,
                             tests_priority_upper_limit,
                             tests_regexp,
+                            build_start_datetime,
+                            build_duration_secs,
                         )
                         if is_actionable_pr:
                             if contains_regression_comment:

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/redis_benchmarks_specification/__common__/github.py

@@ -47,7 +47,7 @@ def generate_build_finished_pr_comment(
     tests_priority_lower_limit,
     tests_priority_upper_limit,
     tests_regexp,
-    build_datetime,
+    build_start_datetime,
     build_duration_seconds,
 ):
     build_duration_seconds = int(build_duration_seconds)
@@ -55,7 +55,7 @@ def generate_build_finished_pr_comment(
     comment_body += (
         "This comment was automatically generated given a benchmark was triggered.\n"
     )
-    comment_body += f"Started building at {build_datetime} and took {build_duration_seconds} seconds.\n"
+    comment_body += f"Started building at {build_start_datetime} and took {build_duration_seconds} seconds.\n"
     comment_body += "You can check each build/benchmark progress in grafana:\n"
     comment_body += f"   - git hash: {git_hash}\n"
     comment_body += f"   - git branch: {git_branch}\n"

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/redis_benchmarks_specification/__self_contained_coordinator__/self_contained_coordinator.py

@@ -1011,31 +1011,46 @@ def process_self_contained_coordinator_stream(
                                     )
 
                                 dataset_load_duration_seconds = 0
+                                try:
+                                    exporter_datasink_common(
+                                        benchmark_config,
+                                        benchmark_duration_seconds,
+                                        build_variant_name,
+                                        datapoint_time_ms,
+                                        dataset_load_duration_seconds,
+                                        datasink_conn,
+                                        datasink_push_results_redistimeseries,
+                                        git_branch,
+                                        git_version,
+                                        metadata,
+                                        redis_conns,
+                                        results_dict,
+                                        running_platform,
+                                        setup_name,
+                                        setup_type,
+                                        test_name,
+                                        tf_github_org,
+                                        tf_github_repo,
+                                        tf_triggering_env,
+                                        topology_spec_name,
+                                        default_metrics,
+                                    )
+                                    r.shutdown(save=False)
+
+                                except redis.exceptions.ConnectionError as e:
+                                    logging.critical(
+                                        "Some unexpected exception was caught during metric fetching. Skipping it..."
+                                    )
+                                    logging.critical(
+                                        f"Exception type: {type(e).__name__}"
+                                    )
+                                    logging.critical(f"Exception message: {str(e)}")
+                                    logging.critical("Traceback details:")
+                                    logging.critical(traceback.format_exc())
+                                    print("-" * 60)
+                                    traceback.print_exc(file=sys.stdout)
+                                    print("-" * 60)
 
-                                exporter_datasink_common(
-                                    benchmark_config,
-                                    benchmark_duration_seconds,
-                                    build_variant_name,
-                                    datapoint_time_ms,
-                                    dataset_load_duration_seconds,
-                                    datasink_conn,
-                                    datasink_push_results_redistimeseries,
-                                    git_branch,
-                                    git_version,
-                                    metadata,
-                                    redis_conns,
-                                    results_dict,
-                                    running_platform,
-                                    setup_name,
-                                    setup_type,
-                                    test_name,
-                                    tf_github_org,
-                                    tf_github_repo,
-                                    tf_triggering_env,
-                                    topology_spec_name,
-                                    default_metrics,
-                                )
-                                r.shutdown(save=False)
                                 test_result = True
                                 total_test_suite_runs = total_test_suite_runs + 1
 
@@ -1054,10 +1069,20 @@ def process_self_contained_coordinator_stream(
                                     logging.critical("Printing redis container log....")
 
                                     print("-" * 60)
-
-                                    print(
-                                        redis_container.logs(stdout=True, stderr=True)
-                                    )
+                                    try:
+                                        print(
+                                            redis_container.logs(
+                                                stdout=True, stderr=True
+                                            )
+                                        )
+                                    except docker.errors.NotFound:
+                                        logging.info(
+                                            "When trying to stop DB container with id {} and image {} it was already stopped".format(
+                                                redis_container.id,
+                                                redis_container.image,
+                                            )
+                                        )
+                                    pass
 
                                     print("-" * 60)
 

{redis_benchmarks_specification-0.1.88 → redis_benchmarks_specification-0.1.90}/redis_benchmarks_specification/test-suites/memtier_benchmark-1Mkeys-string-get-2MB.yml

@@ -9,7 +9,7 @@ dbconfig:
   preload_tool:
     run_image: redislabs/memtier_benchmark:edge
     tool: memtier_benchmark
-    arguments: '"--data-size" "2000000" "--ratio" "1:0" "--key-pattern" "P:P" "-c" "50" "-t" "2" "--hide-histogram" "--key-minimum" "1"'
+    arguments: '"--data-size" "2000000" "--ratio" "1:0" "--key-pattern" "P:P" "-c" "50" "-t" "2" "--hide-histogram" "--key-minimum" "1" "--key-minimum" "1000"'
   resources:
     requests:
       memory: 2g
@@ -22,7 +22,7 @@ build-variants:
 clientconfig:
   run_image: redislabs/memtier_benchmark:edge
   tool: memtier_benchmark
-  arguments: --data-size 2000000  --ratio 0:1 --key-pattern R:R -c 25 -t 4 --hide-histogram --test-time 180
+  arguments: --data-size 2000000 --key-minimum 1 --key-maximum 1000 --ratio 0:1 --key-pattern R:R -c 25 -t 4 --hide-histogram --test-time 180
   resources:
     requests:
       cpus: '4'