Well It seem the testInstallSnapshotDuringBootstrap tests still fails in master ci[1].
Maybe we should reconsider this pr[2] to make it more rubust. [1] https://github.com/apache/ratis/actions/runs/17846245976/job/50745980974 [2] https://github.com/apache/ratis/pull/1275 Best ------------------------- Xinyu Tan On 2025/09/19 02:07:30 Xinyu Tan wrote: > Hi, Tsz Wo > > Thanks for fixing this ci, I has passed all ci in pr[1] and I have merged it! > > I'll continue the release process soon. > > [1] https://github.com/apache/ratis/pull/1284 > > Best > ------------- > Xinyu Tan > > On 2025/09/18 22:55:51 Tsz-Wo Nicholas Sze wrote: > > Hi Attila, > > > > Just tried a 10x10 run for > > TestInstallSnapshotNotificationWithGrpc#testInstallSnapshotDuringBootstrap, > > it passed all of them; see [1]. RATIS-2330 or other JIRAs might have > > fixed it. > > > > Tsz-Wo > > [1] https://github.com/szetszwo/ratis/actions/runs/17841669299 > > > > On Thu, Sep 18, 2025 at 9:50 AM Attila Doroszlai <adorosz...@apache.org> > > wrote: > > > > > > > [1] https://github.com/apache/ratis/pull/1284 > > > > The test failure probably was due to RATIS-2330 > > > > > > There were two failures in the run: > > > - TestRaftServerWithGrpc: RATIS-2330 (fixed, thanks Tsz-Wo) > > > - TestInstallSnapshotNotificationWithGrpc: RATIS-2261 (still open) > > > > > > -Attila > > > > > >
