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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: redis-benchmarks-specification
-Version: 0.1.90
+Version: 0.1.91
 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.90 → redis_benchmarks_specification-0.1.91}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redis-benchmarks-specification"
-version = "0.1.90"
+version = "0.1.91"
 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.90 → redis_benchmarks_specification-0.1.91}/redis_benchmarks_specification/__common__/github.py

@@ -222,6 +222,7 @@ def generate_benchmark_started_pr_comment(
     benchmark_stream_id,
     total_pending,
     total_benchmarks,
+    total_failed,
 ):
     comment_body = (
         "### CE Performance Automation : step 2 of 2 (benchmark) RUNNING...\n\n"
@@ -231,12 +232,15 @@ def generate_benchmark_started_pr_comment(
     )
 
     completed = total_benchmarks - total_pending
+    successful = completed - total_failed
     comment_body += (
         f"Status: {markdown_progress_bar(completed,total_benchmarks,80)} completed.\n\n"
     )
     comment_body += f"In total will run {total_benchmarks} benchmarks.\n"
     comment_body += f"    - {total_pending} pending.\n"
-    comment_body += f"    - {completed} completed.\n"
+    comment_body += f"    - {completed} completed:\n"
+    comment_body += f"      - {successful} successful.\n"
+    comment_body += f"      - {total_failed} failed.\n"
 
     if not isinstance(benchmark_stream_id, str):
         benchmark_stream_id = benchmark_stream_id.decode()

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

@@ -531,7 +531,9 @@ def process_self_contained_coordinator_stream(
 
                 stream_test_list_pending = f"ci.benchmarks.redis/ci/redis/redis:benchmarks:{stream_id}:{running_platform}:tests_pending"
                 stream_test_list_running = f"ci.benchmarks.redis/ci/redis/redis:benchmarks:{stream_id}:{running_platform}:tests_running"
+                stream_test_list_failed = f"ci.benchmarks.redis/ci/redis/redis:benchmarks:{stream_id}:{running_platform}:tests_failed"
                 stream_test_list_completed = f"ci.benchmarks.redis/ci/redis/redis:benchmarks:{stream_id}:{running_platform}:tests_completed"
+
                 for test_file in testsuite_spec_files:
                     if defaults_filename in test_file:
                         continue
@@ -573,8 +575,9 @@ def process_self_contained_coordinator_stream(
                         )
                     filtered_test_files.append(test_file)
                 pending_tests = len(filtered_test_files)
+                failed_tests = 0
                 comment_body = generate_benchmark_started_pr_comment(
-                    stream_id, pending_tests, len(filtered_test_files)
+                    stream_id, pending_tests, len(filtered_test_files), failed_tests
                 )
                 # update on github if needed
                 if is_actionable_pr:
@@ -1125,12 +1128,21 @@ def process_self_contained_coordinator_stream(
                     conn.lrem(stream_test_list_running, 1, test_name)
                     conn.lpush(stream_test_list_completed, test_name)
                     conn.expire(stream_test_list_completed, REDIS_BINS_EXPIRE_SECS)
+                    if test_result is False:
+                        conn.lpush(stream_test_list_failed, test_name)
+                        failed_tests = failed_tests + 1
+                        logging.warning(
+                            f"updating key {stream_test_list_failed} with the failed test: {test_name}. Total failed tests {failed_tests}."
+                        )
                     pending_tests = pending_tests - 1
 
                     # update on github if needed
                     if is_actionable_pr:
                         comment_body = generate_benchmark_started_pr_comment(
-                            stream_id, pending_tests, len(filtered_test_files)
+                            stream_id,
+                            pending_tests,
+                            len(filtered_test_files),
+                            failed_tests,
                         )
                         update_comment_if_needed(
                             auto_approve_github,