lhotari opened a new pull request, #25677: URL: https://github.com/apache/pulsar/pull/25677
### Motivation The shared test BookKeeper clusters used by `SharedPulsarCluster` and `SharedMultiBrokerPulsarCluster` rely on BookKeeper's default `diskUsageThreshold` (0.95). When a developer's local disk is more than 95% full, all bookies are flagged as read-only on startup, and tests that need writable bookies fail with the obscure error `Not enough non-faulty bookies available`. The disk-usage check is irrelevant to the behavior these tests are exercising, so the threshold should be effectively disabled to keep the tests environment-agnostic. ### Modifications - In `SharedPulsarCluster` and `SharedMultiBrokerPulsarCluster`, set `diskUsageThreshold` to `0.999` and `diskUsageWarnThreshold` to `0.99` on the BookKeeper server configuration so the check no longer fails tests on machines with nearly-full disks. ### Verifying this change - [x] Make sure that the change passes the CI checks. This change is a trivial rework / code cleanup without any test coverage. ### Does this pull request potentially affect one of the following parts: - [ ] Dependencies (add or upgrade a dependency) - [ ] The public API - [ ] The schema - [ ] The default values of configurations - [ ] The threading model - [ ] The binary protocol - [ ] The REST endpoints - [ ] The admin CLI options - [ ] The metrics - [ ] Anything that affects deployment -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
