Compatibility with older livestatus releases

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

Compatibility with older livestatus releases #117

Closed llange closed 3 years ago

llange commented 3 years ago

Detect peer support for the following columns, instead of requesting them unconditionally:

services.check_freshness
hosts.event_handler
(services|hosts).staleness

Resolves #116

llange commented 3 years ago

I just added the necessary changes to make the citest pass.

I just don't understand exactly the change I had to apply to table.go.

Should I also add the other flags I added in this PR (HasCheckFreshnessColumn and HasEventHandlerColumn) ? I doesn't change the output of the tests.

sni commented 3 years ago

I'll have a look. The change to table.go should not be required. I need to rework the example/test data, that's a mess right now.

sni commented 3 years ago

thanks, i reworked optional column handling and test data and manually applied your pull request while doing that.