Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v23.3.x] Health report collection improvements #17864

Conversation

mmaslankaprv
Copy link
Member

Backport of PR #17715

Now the decommission progress api is present in all the nodes

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit cacb97d)
Signed-off-by: Michal Maslanka <michal@redpanda.com>
(cherry picked from commit 5d146b0)
The `get_node_health_request` contained version which wasn't serialized
and therefore it wasn't used in the remote nodes. Removed the unused
version field

Signed-off-by: Michal Maslanka <michal@redpanda.com>
(cherry picked from commit a1e9811)
Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit acca9ec)
Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 7f7484d)
When collecting a remote node report a requester is always querying for
the full report without the filter. We may simplify the logic of report
collection by deprecating the filter field from the request. This will
not influence the backward compatibility as the field is not used anyway
and is always empty.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit ab5a19c)
When node is requesting a local node health report we may use a cached
value instead of collecting a new node health report every time.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 550a448)
…lth`

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 0c4439a)
@mmaslankaprv mmaslankaprv added this to the v23.3.12 milestone Apr 15, 2024
@mmaslankaprv mmaslankaprv linked an issue Apr 15, 2024 that may be closed by this pull request
@mmaslankaprv mmaslankaprv requested review from bharathv and ztlpn April 15, 2024 12:38
@mmaslankaprv
Copy link
Member Author

ci-failure: #8950

@mmaslankaprv mmaslankaprv merged commit 190d797 into redpanda-data:v23.3.x Apr 16, 2024
15 of 18 checks passed
@mmaslankaprv mmaslankaprv deleted the vbotbuildovich/backport-17715-v23.3.x-384 branch April 16, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v23.3.x] Health report collection improvements
2 participants