pnsn / squacapi

Squac repo
MIT License
6 stars 0 forks source link

modify the subject of squac alert emails to include "in"/"out of" #373

Closed alexhutko closed 2 years ago

alexhutko commented 2 years ago

Can we please modify the subject of squac alert emails to include "in"/"out of", i.e. Does this example sound confusing?

SQUAC in alert for 'Upload of waveform based metrics on import05 stations', level 1. SQUAC out of alert for 'Upload of waveform based metrics on import05 stations', level 1.

alexhutko commented 2 years ago

Please clean up body of messages to be more readable.

Current:

Trigger in alert for Monitor: Upload of CN waveform based metrics, Min: 1.0, Max: None, Level: 1 Breaching channels: [{'channel': 'CN:BFSB:--:HHZ', 'count': 0}, {'channel': 'CN:BFSB:--:HNZ', 'count': 0}, {'channel': 'CN:GDR:--:HNZ', 'count': 0}, {'channel': 'CN:GOBB:--:HHZ', 'count': 0}, {'channel': 'CN:GOBB:--:HNZ', 'count': 0}, {'channel': 'CN:NLLB:--:HHZ', 'count': 0}, {'channel': 'CN:OZB:--:HHZ', 'count': 0}, {'channel': 'CN:OZB:--:HNZ', 'count': 0}, {'channel': 'CN:PGC:--:HHZ', 'count': 0}, {'channel': 'CN:PGC:--:HNZ', 'count': 0}, {'channel': 'CN:PNT:--:HHZ', 'count': 0}, {'channel': 'CN:CBB:--:HNZ', 'count': 0}, {'channel': 'CN:HOLB:--:HHZ', 'count': 0}, {'channel': 'CN:HOLB:--:HNZ', 'count': 0}, {'channel': 'CN:LLLB:--:HHZ', 'count': 0}, {'channel': 'CN:LLLB:--:HNZ', 'count': 0}]

Suggestion is to just write out the entire trigger in words.

2022-01-01T08:05 UTC Trigger IN alert for monitor: Upload of CN waveform based metrics Alert if (count) of (hourly_mean) measurements is (outside of) (-5,5) for less than (5) channels in channel group: UW SMAs over the last (5) hours

Breaching channels: CN:BFSB:--:HHZ, count: 0 CN:BFSB:--:HNZ, count: 0 ...

Also change subject: SQUAC alert for 'Upload of NCEDC sniffwave_tally based metrics', level 1

to

SQUAC IN(/OUT OF) alert for 'Upload of NCEDC sniffwave_tally based metrics', level 1