[PR] chore(ci): fix /component-test failing after unrelated pushes [camel]

["gnodet (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] chore(ci): fix /component-test failing after unrelated pushes [camel]

["gnodet (via GitHub)" <gi...@apache.org>]

gnodet merged PR #21903:
URL: https://github.com/apache/camel/pull/21903


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] chore(ci): fix /component-test failing after unrelated pushes [camel]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #21903:
URL: https://github.com/apache/camel/pull/21903#issuecomment-4030272808

   :star2: Thank you for your contribution to the Apache Camel project! :star2:
     :robot: CI automation will test this PR automatically.
   
     :camel: Apache Camel Committers, please review the following items:
   
     * First-time contributors **require MANUAL approval** for the GitHub Actions to run
     * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot although they are normally detected and executed by CI.
     * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR.
     * Build and test logs are available in the summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary.
     
     :warning: Be careful when sharing logs. Review their contents before sharing them publicly.


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org