From 12faf0933195d34c386bc8ce25fac2d55990abe2 Mon Sep 17 00:00:00 2001
From: Jonathan Schwender <55576758+jschwe@users.noreply.github.com>
Date: Sat, 28 Jun 2025 02:21:21 +0200
Subject: [PATCH] CI: Add commit hash to HOS bencher reports (#37757)

Since we don't checkout servo in this job, bencher can't infer the
commit hash and we need to pass it manually.
Bencher documentation for --hash:

https://github.com/bencherdev/bencher/blob/02034b83de9e8f61ab10383002a4fc396ba253cf/services/console/src/chunks/docs-explanation/branch-selection/en/hash.mdx

Testing: No testing. This could cause the HOS CI to fail, but wouldn't
block the MQ. This kind of CI change is quite difficult to test without
a lot of setup overhead. The change itself is trivial though, and the
expected damage would also be low (merge queue would still work).

Issue with the missing commit was:
Reported by @webbeef

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
---
 .github/workflows/ohos.yml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ohos.yml b/.github/workflows/ohos.yml
index 94b7f21a0ca..694938305c4 100644
--- a/.github/workflows/ohos.yml
+++ b/.github/workflows/ohos.yml
@@ -260,4 +260,9 @@ jobs:
           echo "MODEL_NAME=$(hdc bugreport | head -n 20 | grep MarketName | awk '{for (i=2; i<NF; i++) printf $i " "; print $NF}' -)" >> $GITHUB_ENV
       - name: Uploading to bencher.dev
         run: |
-          bencher run --adapter json --file bench.json --project '${{ env.BENCHER_PROJECT }}' --token '${{ secrets.BENCHER_API_TOKEN }}' --github-actions '${{ secrets.GITHUB_TOKEN }}' --testbed="$MODEL_NAME"
+          bencher run --adapter json --file bench.json \
+                      --hash '${{ github.sha }}' \
+                      --project '${{ env.BENCHER_PROJECT }}' \
+                      --token '${{ secrets.BENCHER_API_TOKEN }}' \
+                      --github-actions '${{ secrets.GITHUB_TOKEN }}' \
+                      --testbed="$MODEL_NAME"