Merge pull request #707 from Altinity/download_correct_build_log

MyroTk · web-flow · commit cbab6f85bf85 · 2025-04-02T09:37:05.000-04:00
CI/CD Set build log download to current PR
diff --git a/tests/ci/ci.py b/tests/ci/ci.py
@@ -295,6 +295,8 @@ def _pre_action(s3, job_name, batch, indata, pr_info):
         #   testing), otherwise reports won't be found
         if not (pr_info.is_scheduled or pr_info.is_dispatched):
             report_prefix = Utils.normalize_string(pr_info.head_ref)
+    elif pr_info.is_pr:
+        report_prefix = str(pr_info.number)
     print(
         f"Use report prefix [{report_prefix}], pr_num [{pr_info.number}], head_ref [{pr_info.head_ref}]"
     )

Original file line number	Diff line number	Diff line change
`@@ -295,6 +295,8 @@ def _pre_action(s3, job_name, batch, indata, pr_info):`
`295`	`295`	`# testing), otherwise reports won't be found`
`296`	`296`	`if not (pr_info.is_scheduled or pr_info.is_dispatched):`
`297`	`297`	`report_prefix = Utils.normalize_string(pr_info.head_ref)`
	`298`	`+ elif pr_info.is_pr:`
	`299`	`+ report_prefix = str(pr_info.number)`
`298`	`300`	`print(`
`299`	`301`	`f"Use report prefix [{report_prefix}], pr_num [{pr_info.number}], head_ref [{pr_info.head_ref}]"`
`300`	`302`	`)`