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 process/thread count summary to Busy page #4806

Merged
merged 6 commits into from Feb 19, 2021
Merged

Add process/thread count summary to Busy page #4806

merged 6 commits into from Feb 19, 2021

Conversation

mperham
Copy link
Collaborator

@mperham mperham commented Feb 9, 2021

Sadly my design skills are minimal so it's not very attractive. CSS help appreciated if anyone wants to polish.

Screen Shot 2021-02-09 at 3 49 51 PM

@mperham
Copy link
Collaborator Author

mperham commented Feb 18, 2021

Here's another option for UI: a new "Cluster Summary" area at the top:

Screen Shot 2021-02-18 at 10 22 19 AM

Yet another option:

Screen Shot 2021-02-18 at 10 32 11 AM

@mperham
Copy link
Collaborator Author

mperham commented Feb 18, 2021

Latest

Screen Shot 2021-02-18 at 1 22 55 PM

@sjahu
Copy link
Contributor

sjahu commented Feb 18, 2021

🎉 latest version using stats-container/stat looks good!

Another simple approach would be just adding a "total" row to the existing table, since all the stats are derived from it anyway.
image
Totally depends on how prominent you want the summary to be, though; your way's probably better!

@mperham
Copy link
Collaborator Author

mperham commented Feb 18, 2021

I thought about that. This change request comes from someone running hundreds of Sidekiq processes so finding the end of that table is actually tough, that’s why I was trying to put this info at the top. 🤷‍♂️

@sjahu
Copy link
Contributor

sjahu commented Feb 18, 2021

Makes sense 👍

@mperham
Copy link
Collaborator Author

mperham commented Feb 18, 2021

I think I will add a sum of RSS as another stat to fill out the row; that might help people detect memory changes.

This was referenced Mar 16, 2021
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