search

sni / lmd

Livestatus Multitool Daemon - Create livestatus federation from multiple sources
https://labs.consol.de/omd/packages/lmd/
GNU General Public License v3.0
42 stars 31 forks source link

Revert "Revert "Add service info columns to the host table (#58)" (#60)" #62

Closed jacobbaungard closed 5 years ago

jacobbaungard commented 5 years ago

This reverts commit a9525f1243ad7f9f3a5e25424c0f60d1a12e5cf8 (which reverts the service info columns commit).

The actually issue we had was not related to this commit, but due to incompatibilities between our naemon-livestatus and LMD (dependency columns). Therefore this commit can safely be included again.

Apologies for the confusion.

sni commented 5 years ago

are you sure? ;)

jacobbaungard commented 5 years ago

Yes, we (myself and @jimorie) verified that the issue we had was not actually related to this commit at all, but rather due to the dependency columns added in LMD paired with an outdated version of livestatus.

Also verified this works with both new and older livestatus given the earlier commit to only pull dependency columns on Naemon 1.0.10.

jacobbaungard commented 5 years ago

Thanks, and sorry again for the confusion :-)

sni commented 5 years ago

no problem, don't worry :)