search

shinken-monitoring / mod-logstore-mongodb

Shinken module for exporting logs to mongodb from the Livestatus module
GNU Affero General Public License v3.0
5 stars 10 forks source link

Invalid log lines in Info log ... #5

Open mohierf opened 10 years ago

mohierf commented 10 years ago

Invalid logs lines should make a debug log ... not an info log !

2014-03-09 08:11:18,683 [1394349078] Info : [LogStoreMongoDB] This line is invalid: [1394349075] TIMEPERIOD TRANSITION: 24x7;-1;1
2014-03-09 08:11:18,717 [1394349078] Debug : [Livestatus Log Lines] Does not match
2014-03-09 08:11:18,717 [1394349078] Info : [LogStoreMongoDB] This line is invalid: [1394349075] TIMEPERIOD TRANSITION: none;-1;0
2014-03-09 08:11:18,752 [1394349078] Debug : [Livestatus Log Lines] Does not match
2014-03-09 08:11:18,752 [1394349078] Info : [LogStoreMongoDB] This line is invalid: [1394349075] TIMEPERIOD TRANSITION: workhours;-1;0
2014-03-09 08:11:18,787 [1394349078] Debug : [Livestatus Log Lines] Does not match
2014-03-09 08:11:18,787 [1394349078] Info : [LogStoreMongoDB] This line is invalid: [1394349075] TIMEPERIOD TRANSITION: us-holidays;-1;0
2014-03-09 08:11:18,822 [1394349078] Debug : [Livestatus Log Lines] Does not match
Seb-Solon commented 10 years ago

Well, it's a point of view :). Invalid is something wrong so you may say it to the user.

mohierf commented 10 years ago

In fact, you may choose:

It is also a point of view ;-)

In fact, logstore-mongo makes a log in brokerd.log each time you write a log line in shinken.log because this module cannot store this log in mongo ! Morevover, as far as I understand, it is the normal behaviour not to store this log ... In this case, I prefer having a debug log or, even better, no log at all ...