Hi Yong,
Thanks for finding this critical issue. I think it's dangerous to
merge new PRs before the CI is fixed, because we can't ensure whether
the new PRs introduces bugs.
Thanks,
Hang
Yong Zhang <zhangyong1025...@gmail.com> 于2023年2月21日周二 09:15写道：
>
> Hi folks,
>
> I found our CI may have some problems.
>
> In the master code, the tests failed, but the CI was successful.
> The following is the latest commit CI action:
> https://github.com/apache/bookkeeper/actions/runs/4219246330/jobs/7324432945
> Error: Tests run: 633, Failures: 0, Errors: 29, Skipped: 0, Flakes: 1
>
> You can see there have errors but the CI passed.
>
> Also, the PR's check has the same issue.
> https://github.com/apache/bookkeeper/actions/runs/4217320319/jobs/7321051845
>
> We may need to fail the CI if there have errors.
>
> Should we stop merging PRs until we fix the CI? But looks like most
> failures are
> about bookie shell commands. I am not sure whether we need to stop merging.
>
> Best,
> Yong
