sensu / sensu-go

Simple. Scalable. Multi-cloud monitoring.
https://sensu.io
MIT License
1.02k stars 176 forks source link

remove redundant OPC CheckIn #4978

Closed c-kruse closed 1 year ago

c-kruse commented 1 year ago

Not totally sure what was going on here. It appears that there is a redundant CheckIn operation in handleUpdate. Stats from the performance testing cluster's postgres instance show that it is spending more time executing this statement than any others at baseline (many agents connected, but not processing significant event load.) This should cut that baseline in half 😄, although I suspect it will still be at or near the top.

First CheckIn immediately preceding calling handleUpdate: https://github.com/sensu/sensu-go/blob/main/backend/keepalived/keepalived.go#L283-L317