kafbat / kafka-ui

Open-Source Web UI for managing Apache Kafka clusters
http://ui.docs.kafbat.io
Apache License 2.0
625 stars 81 forks source link

Don't show active controller for KRaft clusters #525

Open bachmanity1 opened 3 months ago

bachmanity1 commented 3 months ago

Issue submitter TODO list

Is your proposal related to a problem?

No response

Describe the feature you're interested in

For clusters running in KRaft mode, the describe cluster API currently returns any alive broker's ID as controller ID Link to code. I believe this was done because, before KIP-919, clients couldn’t directly communicate with controllers and had to connect through brokers, making the specific broker ID irrelevant. Given this, perhaps for KRaft clusters, it might be better not to display the active controller indicator (and the green tick in front of the broker ID) to avoid confusing users?

Alternatively, in addition to the broker list, it might be helpful to also display the controller list by calling the describe cluster API with EndpointType set to 2. Or perhaps this could be displayed as a single list, since some nodes may have both broker and controller roles. In this scenario, displaying the active controller would make sense.

Describe alternatives you've considered

No response

Version you're running

1.0.0

Additional context

No response

bachmanity1 commented 3 months ago

@Haarolean, do you plan to remove the active controller from the UI for KRaft clusters, or do you plan to include the controller nodes as well?

Haarolean commented 1 month ago

@Haarolean, do you plan to remove the active controller from the UI for KRaft clusters, or do you plan to include the controller nodes as well?

no idea yet, maybe both