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

Implement More CPU Percentage Color Indicators #2995 #3035

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

Tacitor
Copy link
Contributor

@Tacitor Tacitor commented Nov 28, 2024

Description

Please describe the goal of this pull request.

This PR is adding a new option to the config file

[processlist]
username_warning=root

And then performs and equality check in glances/plugins/processlist/__init__.py to see if there is any process with the same username and displays it with a CRITICAL decoration.

This PR also digs into the CPU percentage colour indicators and found some issues with the get_alert() function. There seems to be an issue with all non critical limits. Any warning or careful limit is not being used correctly in get_alert(). As I result I created a new function to bypass get_alert() for now so that #2995 is fully fixed and implemented.

I also had some issues with the linter and generating docs so I am sorry those parts are a little neglected in this PR.

Resume

@nicolargo
Copy link
Owner

Note: Implementation should also be done in the WebUI

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