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

Absolute thresholds #10

Closed
wants to merge 4 commits into from
Closed

Absolute thresholds #10

wants to merge 4 commits into from

Conversation

c0xc
Copy link

@c0xc c0xc commented Apr 14, 2014

I hacked a little to make it accept absolute threshold values. Integers/floats will still be interpreted as percentages, but 500M would be an absolute threshold.

Also, the program will not accept bogus input anymore like this:
$ ./check_mem.pl -f -C -w 99.999 -c 1monkeypoop
Argument "1monkeypoop" isn't numeric in numeric eq (==) at ./check_mem.pl line 318.
WARNING - 26.2% (2145244 kB) free!|TOTAL=8177544KB;;;; USED=6032300KB;81;8095768;; FREE=2145244KB;;;; CACHES=1807308KB;;;;

Hope this will be useful for someone.

@gabrielcocenza
Copy link
Contributor

Is there a reason why this PR wasn't reviewed?
I think it's a great feature adding absolute threshold values.

@justintime
Copy link
Owner

I think it just got lost in the shuffle. I don't even manage a nagios install any more, so I can't really test it. If you want to fix the merge conflicts and test it out a bit, I'll merge it.

@justintime
Copy link
Owner

Implemented as part of #27

@justintime justintime closed this Sep 26, 2022
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