karmi / monittr

Ruby and web interface for Monit statistics
http://ephemera.karmi.cz/post/1714578431/monittr-ruby-and-web-interface-for-multiple-monits
MIT License
237 stars 33 forks source link

Adding more parameters to services #6

Open hron84 opened 11 years ago

hron84 commented 11 years ago

Monit's xml contains more parameters than we already support, it would be nice if we can support more info about system and services.

New parameters:

I added tests for these values too, and fixed formatting of source where it is needed. All tests are passing

hron84 commented 11 years ago

For pass tests on Travis, see #5 too.

karmi commented 11 years ago

Are you sure all the parameters are available on current/most used Monit versions? I know there are some incompatibilities/inconsistencies...

hron84 commented 11 years ago

@karmi I checked it against Debian Lenny / Monit 4.10.1.

Filesystem: inode percent values are missing Process: big total values are missing System: all //platform related values are missing.

However I thinking on two thing:

My point is offer all new things - if somebody has an older monit, he could get nil - but nil is not an exception. I think clients can handle nils same way as we handle them in the sinatra application (displaying N/A).

I think current distributions does not provide monit older than 5.2 - and these things are supported quite long time ago. However, if anyone wants newer monit - he can recompile monit from source or grab TildeSlash binaries.

But you know what do you want... this is just my 2 cents.