ci: post PR E2E checks with subprocess.run

The reporter used subprocess.check_call(..., input=...), which Popen
rejects. Placeholders made that path run immediately, so the publisher
exited before creating any pull request checks.

Signed-off-by: Zujian Zhang <zhangzujian.7@gmail.com>

Author: zhangzujian

.github/workflows/build-x86-image.yaml

@@ -5488,7 +5488,7 @@ jobs:
                       None,
                   )
                   if checkId is None:
-                      subprocess.check_call(
+                      subprocess.run(
                           [
                               "gh",
                               "api",
@@ -5500,12 +5500,13 @@ jobs:
                           ],
                           input=json.dumps(payload),
                           text=True,
+                          check=True,
                       )
                   else:
                       update = dict(payload)
                       update.pop("name", None)
                       update.pop("head_sha", None)
-                      subprocess.check_call(
+                      subprocess.run(
                           [
                               "gh",
                               "api",
@@ -5517,6 +5518,7 @@ jobs:
                           ],
                           input=json.dumps(update),
                           text=True,
+                          check=True,
                       )
                   published[externalId] = fingerprint
                   print(

hack/test_e2e_control.py

@@ -1495,6 +1495,10 @@ def testPullRequestsExecuteAutomaticCoverageAndPushStaysFull(self):
         self.assertIn("prCheckRunsToPublish", publish)
         self.assertIn("visibleInfrastructureTitles", publish)
         self.assertIn("selectedExecutorJobsAreTerminal", publish)
+        self.assertIn("subprocess.run(", publish)
+        self.assertIn("input=json.dumps(payload)", publish)
+        self.assertIn("check=True", publish)
+        self.assertNotIn("subprocess.check_call", publish)
         self.assertIn("time.sleep(20)", publish)
         self.assertIn("HEAD_SHA: ${{ inputs.headSHA }}", publish)
         self.assertIn("- e2e-selection\n", publish)