From 5166c30e29994fd459705393490e38f2e9192f39 Mon Sep 17 00:00:00 2001 From: Yih-Dar <2521628+ydshieh@users.noreply.github.com> Date: Fri, 21 Apr 2023 20:36:35 +0200 Subject: [PATCH] Fix a minor bug in CI slack report (#22906) * fix * fix --------- Co-authored-by: ydshieh --- utils/get_previous_daily_ci.py | 2 +- utils/notification_service.py | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/utils/get_previous_daily_ci.py b/utils/get_previous_daily_ci.py index 0bcb7fc890..4e4cb0a8c1 100644 --- a/utils/get_previous_daily_ci.py +++ b/utils/get_previous_daily_ci.py @@ -57,9 +57,9 @@ def get_last_daily_ci_reports(artifact_names, output_dir, token): results = {} for artifact_name in artifact_names: - results[artifact_name] = {} artifact_zip_path = os.path.join(output_dir, f"{artifact_name}.zip") if os.path.isfile(artifact_zip_path): + results[artifact_name] = {} with zipfile.ZipFile(artifact_zip_path) as z: for filename in z.namelist(): if not os.path.isdir(filename): diff --git a/utils/notification_service.py b/utils/notification_service.py index 78ad45df39..9fac01c637 100644 --- a/utils/notification_service.py +++ b/utils/notification_service.py @@ -423,8 +423,11 @@ class Message: artifact_names=artifact_names, output_dir=output_dir, token=os.environ["ACCESS_REPO_INFO_TOKEN"] ) - # The last run doesn't produce `test_failure_tables` (by some issues or have no model failure at all) - if len(prev_tables) > 0: + # if the last run produces artifact named `test_failure_tables` + if ( + "test_failure_tables" in prev_tables + and "model_failures_report.txt" in prev_tables["test_failure_tables"] + ): # Compute the difference of the previous/current (model failure) table prev_model_failures = prev_tables["test_failure_tables"]["model_failures_report.txt"] entries_changed = self.compute_diff_for_failure_reports(model_failures_report, prev_model_failures)