RATIS-2501. Improve diagnostics for testInstallSnapshotDuringBootstrap timeout failures.

[slfan1989]

What changes were proposed in this pull request?

This PR improves the diagnostics and reliability of InstallSnapshotNotificationTests.testInstallSnapshotDuringBootstrap by adding bounded retry logic and comprehensive cluster state logging when configuration changes fail.

Key changes:

1. Added setConfigurationWithBoundedRetry() method:

• Limits setConfiguration retries to 30 attempts (vs unlimited retries previously)
• Adds 1-second sleep between retry attempts
• Logs each attempt with leader ID and target configuration
• Dumps detailed cluster state on failure

2. Added waitAndCheckNewConfWithDiagnostics() method:

• Wraps RaftServerTestUtil.waitAndCheckNewConf() with exception handling
• Triggers cluster state dump on any assertion or exception

3. Added dumpClusterState() diagnostic method:

• Logs comprehensive cluster information including:
  • Snapshot request/notification counts
  • Leader snapshot info
  • Per-division state (role, leader, term, indices, configuration)
  • Follower next/match indices
  • All server logs

4. Updated `testInstallSnapshotDuringBootstrap():

• Replaced direct cluster.setConfiguration() call with bounded retry version
• Replaced configuration check with diagnostic version

What is the link to the Apache JIRA

JIRA: RATIS-2501. Improve diagnostics for testInstallSnapshotDuringBootstrap timeout failures.

Please replace this section with the link to the Apache JIRA)

How was this patch tested?

• Existing unit tests

[slfan1989]

@szetszwo I’ve recently been working on improvements related to gRPC zero-copy and have spent some time reviewing and understanding the relevant code. Based on the progress so far, I think it makes sense to continue moving this work forward, and I plan to keep refining this area over the coming period.

Previously, I submitted change #1417, which mainly added some metrics. However, during testing, some unit tests failed due to timeouts, while the same tests passed when run locally. Given this situation, I’m planning to submit a new PR to improve the error messages in the unit tests.

Could you mind taking a look at this change? Thank you very much!

[szetszwo]

@slfan1989 , thanks a lot for fixing gRPC zero-copy!

+1 the change looks good.

[slfan1989]

@slfan1989 , thanks a lot for fixing gRPC zero-copy!

+1 the change looks good.

@szetszwo Thank you so much for reviewing the code!