Changeset 59571

Timestamp:

01/02/2025 04:39:28 PM (16 months ago)

Author:

flixos90

Message:

Build/Test Tools: Revert [59570].

The fix did not work properly and is causing performance data for commits to not be sent at all. It is therefore reverted until a proper solution has been implemented.

See #62766.

File:

• trunk/.github/workflows/reusable-performance.yml (modified) (3 diffs)

trunk/.github/workflows/reusable-performance.yml

@@ -95 +95 @@
   # - Add workflow summary.
   # - Set the base sha.
+  # - Set commit details.
   # - Publish performance results.
   # - Ensure version-controlled files are not modified or deleted.
@@ -312 +313 @@
             return baseRef.data.object.sha;
 
+      - name: Set commit details
+        # Only needed when publishing results.
+        if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/trunk' && ! inputs.memcached }}
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+        id: commit-timestamp
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            const commit_details = await github.rest.git.getCommit({ owner: context.repo.owner, repo: context.repo.repo, commit_sha: context.sha });
+            return parseInt((new Date( commit_details.data.author.date ).getTime() / 1000).toFixed(0))
+
       - name: Publish performance results
         # Only publish results on pushes to trunk.
@@ -317 +329 @@
         env:
           BASE_SHA: ${{ steps.base-sha.outputs.result }}
-          COMMITTED_AT: $(git show -s $GITHUB_SHA --format="%cI")
+          COMMITTED_AT: ${{ steps.commit-timestamp.outputs.result }}
           CODEVITALS_PROJECT_TOKEN: ${{ secrets.CODEVITALS_PROJECT_TOKEN }}
           HOST_NAME: "www.codevitals.run"