Shell Programming and Scripting

Originally Posted by Don Cragun

Hi vgersh99,
Good catch on the -gt versus > test. Without that change, it could miss reporting that the CPU load was 100% (but it still shouldn't have caused any false high load reports). Note that the script we were shown in post #4 didn't have this bug.

Note the code marked in red above. I agree wholeheartedly that the temp file is not needed (and suggested removing it back in post #5 in this thread), but the temp file is removed before it is appended to in the code you're questioning, so that shouldn't have caused any false high load reports either (assuming the code shown to us in post #9 is the actual code being run).

[/NOPARSE]

I can change and test


Hi anil529,
Why don't you also make the changes I suggested in post #5 in this thread (where I also proposed getting rid of the temp file) and get rid of two unneeded processes that are adding unneeded load to the system you're trying to monitor?

If you decide to try drysdalk's suggestion instead, at least note that you must change the:

Code:

if [[ "$load" > "$threshold" ]]

to:

Code:

if [[ "$load" -gt "$threshold" ]]

as noted above by vgersh99 to keep from missing reports if the load reaches 100%.