Enhancement: Better performance and simpler logic (somehow).

shinken-monitoring / mod-livestatus

Shinken module for presenting data with a MK/Livestatus comptabile interface

GNU Affero General Public License v3.0

15 stars 20 forks source link

Enhancement: Better performance and simpler logic (somehow). #38

Closed gst closed 9 years ago

gst commented 9 years ago

Turned at most everything(most functions returning data) into generators.
Created a LiveStatusThread which handle completely a session.
LiveStatus_broker.manage_lql_thread():

Simplified/Factorized No more busy loop. (a dedicated thread listen for new connections).
Misc simplifications.

There are still cleaning to be done but this is a first usable state.

It only needs shinken master to be updated with following pull request: https://github.com/naparuba/shinken/pull/1348

naparuba commented 9 years ago

some benchs to share? :)

gst commented 9 years ago

Le 2014-10-20 11:02, Gabès Jean a écrit :

some benchs to share? :)

when I've updated our broker to latest code :

broker CPU usage

— Reply to this email directly or view it on GitHub https://github.com/shinken-monitoring/mod-livestatus/pull/38#issuecomment-59771525.

titilambert commented 9 years ago

@naparuba I don't know if you see the @gst answer completely ... On our Shinken server (Arb, Sche, Pol, Bro, Rec, Reac), we see the CPU utilization drops from 80% to 30% ( with 200 hosts/2500 services) :)