On Mon, November 28, 2016 17:29, Tim Kuijsten wrote:
> Op 28-11-16 om 15:11 schreef Kirill Bychkov:
>> On Mon, November 28, 2016 16:51, Tim Kuijsten wrote:
>>> Op 28-11-16 om 14:37 schreef Kirill Bychkov:
>>>> On Mon, November 28, 2016 02:22, Tim Kuijsten wrote:
>>>>> Hi ports@,
>>>>>
>>>>> I've updated cnagios to the latest release. Quite some segfaults have
>>>>> been fixed since 0.30.
>>>>>
>>>>> Tested on amd64.
>>>>>
>>>>> -Tim
>>>>>
>>>>> ps. I'm still looking for an easy way to use it with icinga. Maybe a
>>>>> cnagios package on it's own instead of being a subpackage of nagios
>>>>> would be better. I'm not sure..
>>>>>
>>>>
>>>> Hmm. It tries to detect data version of status.dat on configure stage.
>>>> If there is no file, configure fails.
>>>
>>> I think this is related to "Remove dependencies on live nagios instance
>>> #6" [1].
>>
>> That's it.
>>
>>>
>>>> For my icinga installation it detects version3. This could be overrided
>>>> with CONFIGURE_ARGS = --with-nagios-data=3.
>>>> But what should be set for nagios itself? Any nagios users around?
>>>
>>> I think status file format 3 is the latest in both nagios and icinga.
>>
>> Thanks to sane defaults in OpenBSD packages! It is enough to install nagios
>> and run rcctls start nagios to make it run and collect some data for local
>> host!
>> But cnagios tells me that nagios uses data version 4:
>> [...]
>> nagios var dir... /var/nagios
>> nagios status file... /var/nagios/status.dat
>> nagios data version... 4
>> [...]
>>
>>
>>>
>>> -Tim
>>>
>>> [1] https://github.com/dannywarren/cnagios/issues/6
>>>
>>>>
>>>> WIP diff attached. Currently only icinga FLAVOR could be installed.
>>>> Default flavor will fail if there is no /var/nagios/status.dat.
>>>>
>>>> P.S. Briefly tested with icinga.
>>>>
>>
>> Updated patch is attached.
>> OK to commit?
>>
>
> There is a typo in the diff: *+Upsatream* commit
> 3bd27fb40e68f61ffd01bea6234b919a667b6fe4
>
Thanks! Fixed in my tree.
