bosun-monitor / bosun

Time Series Alerting Framework
http://bosun.org
MIT License
3.4k stars 495 forks source link

Fix duplication of chained notifications while reload config through api #2445

Closed svagner closed 4 years ago

svagner commented 4 years ago

Description

Fixes #2444

Type of change

From the following, please check the options that are relevant.

How has this been tested?

The test is described in issue #2444

Checklist:

svagner commented 4 years ago

I've tested locally and this looks good. Thanks very much @svagner

Thanks @tdinucci! I've changed the author of commit (due to the last check) and made force push. Could you please approve it again?:)

svagner commented 4 years ago

@tdinucci I see that merging is blocked for my account. Are you able to merge it to master branch?