From 05d7ef04613b01565d933ccdfa46d04e3d6165dd Mon Sep 17 00:00:00 2001 From: Vasily Nemkov Date: Fri, 16 Aug 2024 17:36:00 +0000 Subject: [PATCH 1/2] Make sure that build reports that are downloaded belong to a certain PR --- tests/ci/ci.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ci/ci.py b/tests/ci/ci.py index bba20f4806dd..337f8ccde368 100644 --- a/tests/ci/ci.py +++ b/tests/ci/ci.py @@ -1165,7 +1165,7 @@ def _pre_action(s3, indata, pr_info): ci_cache = CiCache(s3, indata["jobs_data"]["digests"]) # for release/master branches reports must be from the same branches - report_prefix = normalize_string(pr_info.head_ref) if pr_info.number == 0 else "" + report_prefix = normalize_string(pr_info.head_ref) if pr_info.number == 0 else str(pr_info.number) print( f"Use report prefix [{report_prefix}], pr_num [{pr_info.number}], head_ref [{pr_info.head_ref}]" ) From fbfad5bbf7f1698c7e51c1131ee12fe55e3c997b Mon Sep 17 00:00:00 2001 From: Vasily Nemkov Date: Fri, 16 Aug 2024 17:39:01 +0000 Subject: [PATCH 2/2] A bit more loging --- tests/ci/ci.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/ci/ci.py b/tests/ci/ci.py index 337f8ccde368..8d296f91806c 100644 --- a/tests/ci/ci.py +++ b/tests/ci/ci.py @@ -628,6 +628,7 @@ def download_build_reports(self, file_prefix: str = "") -> List[str]: ) if file_prefix: path += "_" + file_prefix + logging.info(f"Downloading build reports with prefix '{path}'") reports_files = self.s3.download_files( bucket=S3_BUILDS_BUCKET, s3_path=path,