[v25.1.x] Decommission status improvements #26147
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit 3b20ef2)
Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit f1f97cc)
Introduced a type providing detailed information about reallocation failure. Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit 936098d)
…ations When partition replicas area moved in the cluster from whichever reason Redpanda should expose all information required to identify issues experienced during that operation. Previously partition balancer planner included only the basic information about the failures in the status. That was the list of `ntps` that replicas failed to be reallocated and the total number of reallocations failures. In order to obtain more detailed information looking into the partition balancer log was required. Moreover the reallocations that were not even attempted during preparing an allocation plan were not reported at all. This lead to confusion as f.e. partitions with only one replica on the node that is offline that can not be moved was not reported ans the failed one. In order to address this problem we introduced a new `allocation_failure_details` map in the partition balancer planner result. The map provides a detailed information about the failed reallocation. Additionally reallocation failure is reported when partition is marked as `immutable` in the process of planning balancer action. This makes it easy for the operator to identify any partition balancer issues without the need to look for the specific log entries. The change is fully backward compatible as all the changes are incremental i.e. when no detailed information about failures is available Redpanda will fallback to the old basic `ntp` list. Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit 48834cf)
…atus Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit ba820b0)
Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit 17ff1d9)
Signed-off-by: Michał Maślanka <[email protected]> (cherry picked from commit 8f4b6a4)
Retry command for Build#66007please wait until all jobs are finished before running the slash command |
CI test resultstest results on build#66007
|
piyushredpanda
approved these changes
Jun 11, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of PR #26054