Skip to content

Commit 8fdbd61

Browse files
committed
ci: profiling check run named for the reader, verdict wording in the title
1 parent 072a11e commit 8fdbd61

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

scripts/ci/check-profiling-results.sh

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,11 @@ for var in GH_TOKEN GITHUB_SHA GITHUB_REPOSITORY GITHUB_API_URL; do
6767
[ -n "$val" ] || { echo "ERROR: $var is not set" >&2; exit 2; }
6868
done
6969

70-
check_name='profiling: CEst regression'
70+
# The name stays short and stable; the verdict lives in the title. Check
71+
# runs created with the Actions GITHUB_TOKEN get attached to an arbitrary
72+
# workflow's check suite (GitHub limitation, community discussion #24616),
73+
# so the workflow-name prefix GitHub renders cannot be relied on.
74+
check_name='profiling'
7175

7276
curl -fsSL -o cest-analyzer "$store/dashboard/bin/cest-analyzer"
7377
chmod +x ./cest-analyzer
@@ -86,9 +90,9 @@ if [ -z "$leaf" ] || ! curl -sfI --connect-timeout 5 --max-time 20 -o /dev/null
8690
fi
8791

8892
case "$rc" in
89-
0) conclusion=success; title="No CEst regression" ;;
90-
1) conclusion=neutral; title="CEst regression vs the previous profiling run" ;;
91-
*) conclusion=neutral; title="Profiling gate did not run (analyzer exit $rc)" ;;
93+
0) conclusion=success; title="no performance degradation detected" ;;
94+
1) conclusion=neutral; title="latest results need review, performance degradation detected" ;;
95+
*) conclusion=neutral; title="gate did not run (analyzer exit $rc)" ;;
9296
esac
9397

9498
# GitHub caps output.summary at 65535 characters; trim below the cap.

0 commit comments

Comments
 (0)