Icinga2: Remove deprecated features: statusdata, compatlog, checkresultfilereader, commandpipe

[dnsmichi]

6031

6033

6032

[rippiedoos]

If you remove this feature, could you please remove it properly (or fix it in 2.10). Version 2.10 breaks this feature but it's still working in 2.9.2. It is even in the documentation. If you enable the statusdata-feature in 2.10 the objects.cache-file isn't created anymore.
If you run the command icinga2 object list --type=StatusDataWriter in 2.10 it crashes. In 2.9.2 it returns the proper output.

[dnsmichi]

The above was fixed, on removal it won't influence anything else.

[Al2Klimov]

@lippserd Livestatus which is used by one of our key customers uses the compat logger for historical tables. Shall we care about this?

[Al2Klimov]

CC @chrnie

[Al2Klimov]

To be precise: It uses the logs produced by the compat logger.

[dnsmichi]

@lippserd Livestatus which is used by one of our key customers uses the compat logger for historical tables. Shall we care about this?

We cannot remove that for now, that's why this issue is subject for removal from 2.13.

[chrnie]

compat logger is an important feature for thruk's logfile cache.
https://www.thruk.org/documentation/logfile-cache.html
Finally, the entries are to end up in a database.
If this feature is deleted, it would be nice if there was another possibility, apart from importing the IDO. Since not using the IDO was the initial reason to use livestatus.

[Al2Klimov]

You're intentionally not using the IDO? Why?

And yes, there will be another possibility in v2.12 – IcingaDB.

[lippserd]

We should of course double check other tools using our features. If compatlog is used by thruk, we should not remove it. At least for now. We should get in touch with the thruk devs and work on an alternative before removing their access to reports.

statusdata and checkresultfilereader can be removed.

Did we already deprecate the command pipe?

[Al2Klimov]

Yes, we did.