errose28 opened a new pull request, #10148: URL: https://github.com/apache/ozone/pull/10148
## What changes were proposed in this pull request? TestHDDSUpgrade is not run in CI and has not been run in years. It takes extremely long to run and it enumerates a large amount of failure scenarios that are not representative of actual failure domains, especially now that finalization is going to be a single Ratis request. We should remove this test since it is not adding any value and we are not planning to update it for ZDU. This PR also sets us up to remove all `UpgradeFinalizationExecutor`s. In the new versioning framework, these will not be required. After this PR they are only used within the old upgrade framework itself, so when `BasicUpgradeFinalizer` and similar classes are removed, we can remove the finalization executors with them. ## What is the link to the Apache JIRA HDDS-15126 ## How was this patch tested? CI passing on my [fork](https://github.com/errose28/ozone/actions/runs/25002773349/job/73219632969) (one failure in ratis log parser is unrelated) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
