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

Use shards from health report in Admin API cluster partition results #22695

Merged
merged 10 commits into from
Aug 7, 2024

Conversation

ztlpn
Copy link
Contributor

@ztlpn ztlpn commented Aug 2, 2024

If node-local core assignment is enabled, return core assignments from health report in /v1/cluster/partitions admin API output. These will better reflect reality, instead of obsolete assignments queried from the topic table.

Additionally, do some node_health_report refactoring that enables more efficient querying of node health reports.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.2.x
  • v24.1.x
  • v23.3.x

Release Notes

Improvements

  • Return core assignments from health report in /v1/cluster/partitions admin API output.

@ztlpn
Copy link
Contributor Author

ztlpn commented Aug 2, 2024

/ci-repeat

@ztlpn ztlpn force-pushed the flex-assignment-shards-in-admin-api branch from 58d11f7 to 8090519 Compare August 2, 2024 13:21
@ztlpn
Copy link
Contributor Author

ztlpn commented Aug 2, 2024

/ci-repeat

@ztlpn ztlpn force-pushed the flex-assignment-shards-in-admin-api branch from 8090519 to d05491b Compare August 2, 2024 17:34
@ztlpn ztlpn force-pushed the flex-assignment-shards-in-admin-api branch from ca7fbd5 to 1c94bcd Compare August 5, 2024 10:36
@ztlpn ztlpn marked this pull request as ready for review August 5, 2024 10:40
@ztlpn ztlpn merged commit 967ed81 into redpanda-data:dev Aug 7, 2024
21 checks passed
@ztlpn ztlpn deleted the flex-assignment-shards-in-admin-api branch August 7, 2024 12:13
@vbotbuildovich
Copy link
Collaborator

/backport v24.2.x

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.

3 participants