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

Add brokerSet id in kafka_cluster_state #1860

Closed
CCisGG opened this issue Jul 13, 2022 · 0 comments · Fixed by #1889
Closed

Add brokerSet id in kafka_cluster_state #1860

CCisGG opened this issue Jul 13, 2022 · 0 comments · Fixed by #1889
Assignees
Labels
functionality A feature request.

Comments

@CCisGG
Copy link
Contributor

CCisGG commented Jul 13, 2022

As cruise control now supports BrokerSetAwareGoal (introduced in #1809), we may want to show the brokerSet id of each topic in the kafka_cluster_state.

Today, in kafka_cluster_state response, there is a section for broker state:

Screen Shot 2022-08-19 at 10 34 02 AM

What we would like to have, is a new column BROKER_SET which shows the broker set id of each broker. The new broker state should look like below:

Screen Shot 2022-08-19 at 10 34 59 AM

Preferably, this column should only show up when the BrokerSetAwareGoal is in the goals list.

This change will help the users to know what is the broker set id of each broker. Without this change, the broker set info will be opaque to users. E.g. after the rebalance happened, there is no way for users to know which broker set that each broker belongs to. This can introduce unnecessarily challenges for debugging cc and inspecting cluster status. Say if a user implements custom rules to assign broker set to brokers, the broker set arrangement would be unknown to the user after goal optimization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functionality A feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant