Skip to content

Commit 371c444

Browse files
ci(ios): include all run ARNs in results aggregation and log download
The two RUN_ARNS aggregation loops were hardcoded to iterate over indices 2..8, so the new Heavy7/Heavy8 runs (RUN_ARN_9, RUN_ARN_10) were silently dropped from the final test-results summary and the Device Farm log download. As a result, Heavy7/Heavy8 failures would not have failed the workflow and their device logs would not have been collected. Iterate up to RUN_COUNT instead, so any future bucket additions are picked up automatically. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 7d15b6f commit 371c444

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/integration-mobile-test-qvac-lib-infer-llamacpp-llm.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1349,9 +1349,9 @@ jobs:
13491349
done
13501350
13511351
RUN_ARNS=("$RUN_ARN_1")
1352-
for i in 2 3 4 5 6 7 8; do
1352+
for i in $(seq 2 "$RUN_COUNT"); do
13531353
eval "arn=\$RUN_ARN_$i"
1354-
if [ "$RUN_COUNT" -ge "$i" ] && [ -n "$arn" ]; then
1354+
if [ -n "$arn" ]; then
13551355
RUN_ARNS+=("$arn")
13561356
fi
13571357
done
@@ -1509,9 +1509,9 @@ jobs:
15091509
mkdir -p "$LOG_DIR"
15101510
15111511
RUN_ARNS=("$RUN_ARN_1")
1512-
for i in 2 3 4 5 6 7 8; do
1512+
for i in $(seq 2 "$RUN_COUNT"); do
15131513
eval "arn=\$RUN_ARN_$i"
1514-
if [ "$RUN_COUNT" -ge "$i" ] && [ -n "$arn" ]; then
1514+
if [ -n "$arn" ]; then
15151515
RUN_ARNS+=("$arn")
15161516
fi
15171517
done

0 commit comments

Comments
 (0)