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 akka cluster status panel #42

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add akka cluster status panel #42

wants to merge 4 commits into from

Conversation

mtsokol
Copy link

@mtsokol mtsokol commented Sep 17, 2020

Hi!

According to @jczuchnowski's guidelines here's PR for introducing akka-cluster status panel showing members from Akka Management API definition.

Here's example (in top right corner):

Additional panel is enabled when --akka-cluster-status <url> option is provided, and as it's optional component to akka tab it introduced few ifs in the code.

It only shows nodeUid and status as more fields made it obscure in my terminal.

WDYT?

@vpavkin
Copy link
Contributor

vpavkin commented Sep 18, 2020

Looks great! 🚀

@mtsokol
Copy link
Author

mtsokol commented Sep 18, 2020

Here's refactor after decision to exclude this to a separate tab. The tab displays all info from Akka Management API definition (except there's no example how roles and unreachable exactly look like so will work on that next week).

Cluster members are fixed as Paragraph widget as List doesn't support multiline item displaying 😕

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

Successfully merging this pull request may close these issues.

None yet

2 participants