diff --git a/utils/get_previous_daily_ci.py b/utils/get_previous_daily_ci.py index 0bcb7fc8900..4e4cb0a8c10 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 bbdd4e996de..b58c0324c5e 100644 --- a/utils/notification_service.py +++ b/utils/notification_service.py @@ -423,8 +423,11 @@ def per_model_sum(model_category_dict): 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)