gnodet opened a new pull request, #21903:
URL: https://github.com/apache/camel/pull/21903

   ## Summary
   
   Fix the `/component-test` workflow's "Retrieve sha" step which was failing 
whenever any push happened to the repository after the comment was posted.
   
   ## Problem
   
   The step compared `head.repo.pushed_at` with the comment timestamp to ensure 
the PR branch hadn't changed after the test was requested. However, 
`head.repo.pushed_at` is the **repo-level** last push timestamp — it changes 
whenever *any* branch is pushed to (e.g., merging another PR to `main`). This 
caused all `/component-test` runs to fail with:
   
   ```
   exit 1
   ```
   
   in the "Retrieve sha" step, since there's almost always a push to the repo 
between when a comment is posted and when the workflow runs.
   
   ## Fix
   
   Replace `head.repo.pushed_at` with the PR's **head commit date** 
(`commit.committer.date`), which accurately reflects when the PR branch itself 
was last updated. This preserves the safety check (don't run tests against code 
that changed after the comment) while not being tripped by unrelated pushes.
   
   ## Impact
   
   This fixes all `/component-test` invocations, which have been effectively 
broken. Recent examples of failures caused by this bug:
   - https://github.com/apache/camel/actions/runs/22895881978 (couchbase)
   - https://github.com/apache/camel/actions/runs/22895882707 (ibmmq)
   - https://github.com/apache/camel/actions/runs/22895883897 (milvus)
   - https://github.com/apache/camel/actions/runs/22895884782 (milvus ppc64le)
   - https://github.com/apache/camel/actions/runs/22895885225 (weaviate)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to