nezhahq / nezha

:trollface: Self-hosted, lightweight server and website monitoring and O&M tool
https://nezha.wiki
Apache License 2.0
7.34k stars 1.28k forks source link

implement notification group #450

Closed uubulb closed 1 month ago

uubulb commented 1 month ago

增加通知组与通知api,通知方式改为与通知组id绑定

uubulb commented 1 month ago

这个写法感觉锁操作有点多,要不换个支持并发的map

naiba commented 1 month ago

这个写法感觉锁操作有点多,要不换个支持并发的map

放内存里是那种频繁读写的数据,如果不是频繁读写可以就放数据库里

哪边的锁操作有点多?

uubulb commented 1 month ago

这个写法感觉锁操作有点多,要不换个支持并发的map

放内存里是那种频繁读写的数据,如果不是频繁读写可以就放数据库里

哪边的锁操作有点多?

这里 singleton/notification.go 加了通知组后几个 map 的读写更多了,因为不能直接从字段取

naiba commented 1 month ago

这里 singleton/notification.go 加了通知组后几个 map 的读写更多了,因为不能直接从字段取

不需要在内存中放这么多东西吧,只存必要的

uubulb commented 1 month ago

这里 singleton/notification.go 加了通知组后几个 map 的读写更多了,因为不能直接从字段取

不需要在内存中放这么多东西吧,只存必要的

NotificationGroup可以换成从数据库读名字

naiba commented 1 month ago

这里 singleton/notification.go 加了通知组后几个 map 的读写更多了,因为不能直接从字段取

不需要在内存中放这么多东西吧,只存必要的

NotificationGroup可以换成从数据库读名字

嗯嗯,不是像 server 列表那种需要频繁读的都可以放到内存

uubulb commented 1 month ago

这里 singleton/notification.go 加了通知组后几个 map 的读写更多了,因为不能直接从字段取

不需要在内存中放这么多东西吧,只存必要的

NotificationGroup可以换成从数据库读名字

嗯嗯,不是像 server 列表那种需要频繁读的都可以放到内存

感觉SendNotification调用的还是比较多的,alert cron monitor都用了,每次都读数据库可能不太好

uubulb commented 1 month ago

listNotification 放在server之后加吧

naiba commented 1 month ago

listNotification 放在server之后加吧

listserver 有了 server 都结束了

naiba commented 1 month ago

refactor 了一下 common handler 跟进下

naiba commented 1 month ago

希望最终状态是都按组去管理,所以所有的绑定关系都放表里面 一个告警组可以包含多个通知方式