From ff429563e9c9b88c459fb80d5176abf42b37858f Mon Sep 17 00:00:00 2001 From: Xin Ye Date: Mon, 7 Apr 2025 11:02:24 -0500 Subject: [PATCH 1/2] Fix a bug where failed_stage can be None in failure stats report --- reframe/frontend/printer.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/reframe/frontend/printer.py b/reframe/frontend/printer.py index 03f4fef80..8a748ea91 100644 --- a/reframe/frontend/printer.py +++ b/reframe/frontend/printer.py @@ -195,6 +195,8 @@ def failure_stats(self, report, global_stats=False): info += f' @{tc["system"]}:{tc["partition"]}+{tc["environ"]}' failed_stage = tc['fail_phase'] + if not failed_stage: + continue failures.setdefault(failed_stage, []) failures[failed_stage].append(info) From 07e902348f1fb04cabbce85b00e4d9b45290efbf Mon Sep 17 00:00:00 2001 From: Xin Ye Date: Mon, 7 Apr 2025 15:15:46 -0500 Subject: [PATCH 2/2] Minor styling update --- reframe/frontend/printer.py | 1 + 1 file changed, 1 insertion(+) diff --git a/reframe/frontend/printer.py b/reframe/frontend/printer.py index 8a748ea91..a2390e39f 100644 --- a/reframe/frontend/printer.py +++ b/reframe/frontend/printer.py @@ -197,6 +197,7 @@ def failure_stats(self, report, global_stats=False): failed_stage = tc['fail_phase'] if not failed_stage: continue + failures.setdefault(failed_stage, []) failures[failed_stage].append(info)