-
Notifications
You must be signed in to change notification settings - Fork 581
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Main process crashes on Json::Encode (third-party/yajl) #6684
Comments
This would indicate a bug in the Json library we're using, yajl. I was just wondering which library we're using here, either from CentOS/EPEL or from our third-party repository. Looking into the package builds, I can see that's the content from
yail is relatively "old", we're already using the latest release 2.1.0. I am reading lloyd/yajl#168 which indicates that this project is not actively maintained anymore. Considering looking into an alternative JSON implementation, granted that we can now use C++11 features in compilers we couldn't for many years. https://github.com/miloyip/nativejson-benchmark#parsing-time |
Hi Guys, File
/var/log/messages looks similar: `icinga2 --version Copyright (c) 2012-2018 Icinga Development Team (https://icinga.com/) System information: Build information: yajl is installed in the following version: `yum info yajl Installed Packages Unfortunately there are no up-to-date packages for yajl provided by centos. If you need furhter information: please feel free to ask. Best wishes from Unterföhring EL |
a colleague gave me a hint that there might be a connection with #6737 |
https://github.com/Icinga/icinga2/tree/feature/replace-json-library holds a first attempt, but feel free to do it differently with the header only library @Al2Klimov - thanks 👍 |
I feared that and hoped it would provide a mechanism for doing that in its own. I don't like our algorithm, I think there are better ones for that. Still, go for it. |
Thanks for the green light. As this will implicitly solve all of our already known UTF8-problems, please assign me the respective issues. |
@miso231 @pimperator 2.11 will drop YAJL and use a header-only JSON library compiled into the binary. The main reasons for picking another library here is that YAJL isn't maintained anymore, last commit in 2015. The linked logrotate issue is different from this unfortunately. |
Icinga2 crashes from time to time (SIGSEGV). It happens approximately when log rotation is happening so I guess it might be related. I'm using distributed setup with HA master and this issue so far occurs only on client nodes.
Current Behavior
Client nodes in 3 level environment crash from time to time. All nodes run version 2.9.1.
Steps to Reproduce (for bugs)
Hard to tell since this issue happens occasionally.
Context
backtrace
/var/log/messages
Let me know if you need more information.
Your Environment
icinga2 --version
):CentOS Linux 7 (Core)
icinga2 feature list
):api mainlog
icinga2 daemon -C
) - I removed apply rule warnings:zones.conf
file (oricinga2 object list --type Endpoint
andicinga2 object list --type Zone
) from all affected nodes.The text was updated successfully, but these errors were encountered: