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

added iostatCpu for evaluation by project #58

Closed
wants to merge 1 commit into from

Conversation

zo3adams
Copy link

I find the output of 'iostat' to be incredibly helpful for quick glance server status - however it is not installed by default on most systems.

This fork adds cpu idle, system, and user loads to the processes dashboard. For me personally this is helpful info, but not sure how your project feels about the inclusion of non-standard tools.

Issuing a pull request just to open discussion - can revise this fork with a method to not impact index.html if iostat not installed on the system, better page style for including what's here, and to add the other helpful info from iostat (disk iops) if project maintainer is interested?

@cyphar
Copy link

cyphar commented Feb 1, 2014

While I cannot speak for the maintainers, it is my understanding that this project intends to be a drop-in tool, with essentially no dependencies (apart from php and a web server).

However, it would be cool to see this project creating additional functionality which is only outputted in the final HTML if the server has the tools needed. This would allow for far more information to be displayed, but the dependencies would be optional rather than mandatory.

Just contributing my $0.02.

@tariqbuilds
Copy link
Owner

@zo3adams , your addition is awesome. Tools like iostat provide a plethora of rich stats which sys admins would love to see. However, it is not a good fit for the project. As @cyphar mentioned, this project is meant to be a drop-in, minimal dependency dashboard. Tomorrow, I plan to list a set of guidelines for the project as well as a rough roadmap which will outline this.

@cyphar , your recommendation makes a lot of sense. I will try to incorporate that style of rendering the dashboard as v1.0 gets closer.

Thanks, guys.

@tariqbuilds tariqbuilds closed this Feb 1, 2014
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.

3 participants