Thanks for reporting the issue Enrico.

The root cause of this is fixed in
https://github.com/apache/pulsar-test-infra/pull/16 .
*It happens for pull requests with more than 25 commits since we are using
shallow clones because of performance reasons. *
*The shallow clone change was made in
https://github.com/apache/pulsar/pull/8394
<https://github.com/apache/pulsar/pull/8394> .*
I also noticed that the solution in diff-only doesn't work for GitHub
Action workflows triggered by a "push" event. That is fixed in the PR too.

I have also made a PR to replace the diff-only action based solution by
using the built in "paths-ignore" feature:
https://github.com/apache/pulsar/pull/9527

Please review the PRs.

BR, Lari

On Mon, Feb 8, 2021 at 12:41 PM Enrico Olivelli <eolive...@gmail.com> wrote:

> Hello,
> today I saw that the GH queue was pretty empty.
> And I also realized that my PRs had been validated super quickly.
>
> I saw this problem, basically CI is no more validating PRs.
>
> The bad thing is that CI is reported as GREEN !
>
> I created this issue in order to track the problem
> https://github.com/apache/pulsar/issues/9526
>
> It happened at least in those two Pull requests:
> https://github.com/apache/pulsar/pull/9448
> https://github.com/apache/pulsar/pull/8590
>
> Please take a deep loop to PR validation before committing any patch
>
> I hope we can fix this issue ASAP
>
> Enrico
>

Reply via email to