Skip to content

Commit 773eccf

Browse files
committed
resolve edge case if both binary and release builds are not cached
1 parent 84e6040 commit 773eccf

File tree

1 file changed

+16
-2
lines changed

1 file changed

+16
-2
lines changed

ci/praktika/native_jobs.py

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -398,9 +398,23 @@ def check_affected_jobs():
398398

399399
affected_artifacts = []
400400
unaffected_jobs_with_artifacts = {}
401-
# NOTE (strtgbb): We always want the build artifacts for our report and regression tests.
401+
all_required_artifacts = set()
402+
403+
# NOTE (strtgbb): We always want these build artifacts for our report and regression tests.
402404
# If we make FinishCIReport and regression tests into praktika jobs, we can remove this.
403-
all_required_artifacts = set(["CH_AMD_RELEASE", "CH_ARM_RELEASE"])
405+
if "CIReport" in workflow.additional_jobs:
406+
all_required_artifacts.update(["CH_AMD_RELEASE", "CH_ARM_RELEASE"])
407+
if (
408+
"Regression" in workflow.additional_jobs
409+
and "regression"
410+
not in workflow_config.custom_data.get("ci_exclude_tags", [])
411+
):
412+
all_required_artifacts.update(["CH_AMD_BINARY"])
413+
if "aarch64" not in workflow_config.custom_data.get(
414+
"ci_exclude_tags", []
415+
):
416+
all_required_artifacts.update(["CH_ARM_BINARY"])
417+
print(f"Including artifacts for custom jobs [{all_required_artifacts}]")
404418

405419
for job in workflow.jobs:
406420
# Skip native Praktika jobs

0 commit comments

Comments
 (0)