Skip to content

Add akka cluster status panel - #42

Open
mtsokol wants to merge 4 commits into
masterfrom
akka-cluster-status
Open

Add akka cluster status panel#42
mtsokol wants to merge 4 commits into
masterfrom
akka-cluster-status

Conversation

@mtsokol

@mtsokol mtsokol commented Sep 17, 2020

Copy link
Copy Markdown

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

vpavkin commented Sep 18, 2020

Copy link
Copy Markdown
Contributor

Looks great! 🚀

@mtsokol
mtsokol force-pushed the akka-cluster-status branch from 1dddd22 to ca04936 Compare September 18, 2020 13:14
@mtsokol

mtsokol commented Sep 18, 2020

Copy link
Copy Markdown
Author

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 😕

@mtsokol
mtsokol requested a review from vpavkin September 18, 2020 13:22
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.

2 participants