fix per node and aggrgate log errors

Author: Bojan131

scripts/print_aggregated_errors.py

@@ -193,19 +193,21 @@ def print_all_errors():
                     publisher_get_error = attempt_data.get('publisher_get_error')
                     non_publisher_get_error = attempt_data.get('non_publisher_get_error')
 
-                    # Show which errors occurred for this attempt
-                    error_types = []
+                    # Show actual error messages for each error type
+                    error_lines = []
                     if publish_error:
-                        error_types.append("publish")
+                        error_lines.append(f"    • publishing — {publish_error}")
                     if query_error:
-                        error_types.append("query")
+                        error_lines.append(f"    • querying — {query_error}")
                     if publisher_get_error:
-                        error_types.append("local get")
+                        error_lines.append(f"    • local get — {publisher_get_error}")
                     if non_publisher_get_error:
-                        error_types.append("remote get")
+                        error_lines.append(f"    • remote get — {non_publisher_get_error}")
 
-                    if error_types:
-                        print(f"  • {ka_label} (attempt {attempt_number}): {', '.join(error_types)} errors")
+                    if error_lines:
+                        print(f"  • {ka_label} (attempt {attempt_number}):")
+                        for line in error_lines:
+                            print(line)
                     else:
                         print(f"  • {ka_label} (attempt {attempt_number}): no errors")
                 else:
@@ -229,8 +231,34 @@ def print_error_for_node():
     if not errors:
         print("  ✅ No errors\n")
     else:
-        for error_key, count in errors.items():
-            print(f"  • {count}x {error_key}")
+        for attempt_key, attempt_data in errors.items():
+            if isinstance(attempt_data, dict) and 'ka_label' in attempt_data:
+                ka_label = attempt_data.get('ka_label', 'Unknown KA')
+                attempt_number = attempt_data.get('attempt', 1)
+                publish_error = attempt_data.get('publish_error')
+                query_error = attempt_data.get('query_error')
+                publisher_get_error = attempt_data.get('publisher_get_error')
+                non_publisher_get_error = attempt_data.get('non_publisher_get_error')
+                
+                error_lines = []
+                if publish_error:
+                    error_lines.append(f"    • publishing — {publish_error}")
+                if query_error:
+                    error_lines.append(f"    • querying — {query_error}")
+                if publisher_get_error:
+                    error_lines.append(f"    • local get — {publisher_get_error}")
+                if non_publisher_get_error:
+                    error_lines.append(f"    • remote get — {non_publisher_get_error}")
+                
+                if error_lines:
+                    print(f"  • {ka_label} (attempt {attempt_number}):")
+                    for line in error_lines:
+                        print(line)
+                else:
+                    print(f"  • {ka_label} (attempt {attempt_number}): no errors")
+            else:
+                count = attempt_data if isinstance(attempt_data, int) else 1
+                print(f"  • {count}x {attempt_key}")
         print()
 
 if __name__ == "__main__":

tests/mainnet/conftest.py

@@ -187,19 +187,21 @@ def avg_time(times):
                     publisher_get_error = attempt_data.get('publisher_get_error')
                     non_publisher_get_error = attempt_data.get('non_publisher_get_error')
 
-                    # Show which errors occurred for this attempt
-                    error_types = []
+                    # Show actual error messages for each error type
+                    error_lines = []
                     if publish_error:
-                        error_types.append("publish")
+                        error_lines.append(f"    • publishing — {publish_error}")
                     if query_error:
-                        error_types.append("query")
+                        error_lines.append(f"    • querying — {query_error}")
                     if publisher_get_error:
-                        error_types.append("local get")
+                        error_lines.append(f"    • local get — {publisher_get_error}")
                     if non_publisher_get_error:
-                        error_types.append("remote get")
+                        error_lines.append(f"    • remote get — {non_publisher_get_error}")
 
-                    if error_types:
-                        print(f"  • {ka_label} (attempt {attempt_number}): {', '.join(error_types)} errors")
+                    if error_lines:
+                        print(f"  • {ka_label} (attempt {attempt_number}):")
+                        for line in error_lines:
+                            print(line)
                     else:
                         print(f"  • {ka_label} (attempt {attempt_number}): no errors")
                 else:

tests/testnet/conftest.py

@@ -188,19 +188,21 @@ def avg_time(times):
                     publisher_get_error = attempt_data.get('publisher_get_error')
                     non_publisher_get_error = attempt_data.get('non_publisher_get_error')
 
-                    # Show which errors occurred for this attempt
-                    error_types = []
+                    # Show actual error messages for each error type
+                    error_lines = []
                     if publish_error:
-                        error_types.append("publish")
+                        error_lines.append(f"    • publishing — {publish_error}")
                     if query_error:
-                        error_types.append("query")
+                        error_lines.append(f"    • querying — {query_error}")
                     if publisher_get_error:
-                        error_types.append("local get")
+                        error_lines.append(f"    • local get — {publisher_get_error}")
                     if non_publisher_get_error:
-                        error_types.append("remote get")
+                        error_lines.append(f"    • remote get — {non_publisher_get_error}")
 
-                    if error_types:
-                        print(f"  • {ka_label} (attempt {attempt_number}): {', '.join(error_types)} errors")
+                    if error_lines:
+                        print(f"  • {ka_label} (attempt {attempt_number}):")
+                        for line in error_lines:
+                            print(line)
                     else:
                         print(f"  • {ka_label} (attempt {attempt_number}): no errors")
                 else: