search

Nextdoor / zkmonitor

Zookeeper Monitoring Daemon
5 stars 4 forks source link

Send separate emails during alerts #29

Closed mikhail closed 9 years ago

mikhail commented 9 years ago

cc: @diranged

diranged commented 9 years ago

:+1: except for my comment. can you dig into it?

mikhail commented 9 years ago

Good point -- will try/catch and log a critical()

On Tue, Jun 9, 2015 at 1:28 PM, Matt W notifications@github.com wrote:

[image: :+1:] except for my comment. can you dig into it?

— Reply to this email directly or view it on GitHub https://github.com/Nextdoor/zkmonitor/pull/29#issuecomment-110495240.

Mikhail Simin, Ph.D Nextdoor http://nextdoor.com/ The Private Social Network for Neighborhoods

mikhail commented 9 years ago

More digging is required. Starting to suspect that my change won't fix the issue.