sdinteractive / sd-serverdiagnostics

Server Connector For MageDiagnostics
1 stars 2 forks source link

Detect if /tmp/magento dir exists #6

Open toddbc opened 8 years ago

toddbc commented 8 years ago

If /tmp/magento:

We should somehow feed this somewhere so we can report it and alert on it. Note that if it's a file or symlink, it may be working around a problem - maybe still worth reporting.

mpchadwick commented 8 years ago

If the use case is alerting magediagnostics is not the right tool. We may be able to use this or else the exec plugin should work

https://github.com/influxdata/telegraf/blob/master/plugins/inputs/filestat/README.md

mpchadwick commented 8 years ago

Sorry - My response here was pretty blunt. To clarify, we're still early in the lifetime of MageDiagnostics, but I've always had it in my mind that alerting should not be in the scope of functionality provided by that tool. I think this is definitely a good idea, but just that it would be better handled by Telegraf -> InfluxDb -> Kapacitor. Let me know if you have concerns, otherwise I suggest we close this issue.