J.C., Thank you again J.C. for reviving the updates for Xymon. I am willing to help update it as well, even though I am now retired and only have a small install of Xymon 4.3.30 at home for my home network. I have contributed to Xymon several times in the past when I had it installed in a large corporate environment before I retired.
J.C. and 4.4-alpha1 testers, I have done a little bit of testing of 4.4-alpha1 on Rocky Linux 8.8 (server and client) and on a busybox Linux install on a NAS. I looked into the gcc 8.5.0 compiler warnings that I got on Rocky Linux, and have corrected them, or silenced the ones that should not be an issue. Attached are two context diff patch files, one to fix the compiler warnings, and one to make a couple updates that I had applied to 4.3.30 previously. My updates fix the busybox Linux client build, and enhance the temperature graph to optionally display both Celsius and Fahrenheit readings.
I have not fully tested 4.4-alpha1 yet, but wanted to get these first
patches released for 4.4-alpha2.
Tom Schmidt
On Wed, Sep 20, 2023 at 2:38?PM J.C. Cleaver <cleaver at terabithia.org> wrote:
Hello all! Just wanted to update everyone on a few things.
Going through the 4.x codebase re-familiarizing myself again with the state of everything, I recalled that I'd actually built a release artifact before development was paused, including a testing RPM that has actually been running unbeknownst on a VM for quite a while. Unfortunately, while validating that compiles from source worked I ran into what looked like showstopper bugs. These weren't showing up in the package, so I had to pore through a lot more code* than I expected to try to figure out where the package was accidentally fixing it.
(*message-sending code, which changes a lot in 4.4 to support compression, TLS, HTTP, and size-prefixed messages)
Fortunately, it ended up being a relatively simple issue (isn't it always?), with easy-enough validation and workaround (given below). Although more fixes could be put in (and I'm sure will be needed!), I believe there's some benefit to having a stable starting point using a build artifact from 2019.
As a result, I'm pleased to announce the first alpha for Xymon 4.4! As this is an alpha release, this is absolutely NOT suitable for production use and is intended mainly for smoketesting by source users. (And I expect there will be smoke.) The release tarball is available now at:
https://sourceforge.net/projects/xymon/files/Xymon/4.4-alpha/xymon-4.4-alpha...
Major improvements are too many to list here but are noted in the Changes and RELEASENOTES files. README documents for compression, TLS, and IPv6 are in progress.
Some of the changes and improvements other than TLS support have been in the Terabithia packages (to some extent) for a while, so there will be less of a delta for those users to this alpha.
For the moment, I'd like to focus on source tarball users and others running xymon from scratch to help fix underlying issues and any smoketest failures, which I expect may crop up particularly on untested non-Linux platforms. Once the lower-hanging fruit is fixed and more patches are upstreamed and cleaned up, I'll release an alpha2 release at which point packages may be released to help with testing iteration and downstream distributors.
Please flag bug reports or patches as related to "alpha1" in the subject line to help keep things clear.
Thank you to everyone in the community for remaining part of it -- and xymon users! -- during this interregnum. Restarting development has been a process, but I'm glad to take this important first step to our next release.
Regards, -jc
*** KNOWN ISSUES -- IMPORTANT ***
The configure script is likely setting XYMONHOME/XYMONCLIENTHOME and XYMSRV/XYMSERVERS/XYMONSERVERS incorrectly, which will prevent xymonlaunch from correctly launching anything. Double check the xymon environment files (xymonserver/xymonclient.cfg) before starting either the server or client.
The tools/xymon-client.default and tools/xymonlaunch files may be placed into /etc/{sysconfig,default}/{xymon-client,xymonlaunch} to more easily set $XYMONSERVERS
Revealed by the above issue is a bug causing a crash in the client (or anywhere) when given an empty string as the destination argument (instead of 0, 127.0.0.1 or no variable set). Don't do this --> [ xymon '' "ping" ]
The client and logfetch binary may be being re-compiled during 'make install'
Compilation may fail if c-ares and at least one compression library is not present (lzo/lz4/zlib are all options). xxHash libraries are also highly recommended as the plan is for xxhash to replace md5 for checksumming functionality. libtirpc may be necessary on some platforms, and it's possible the build code is still expecting traditional (Sun) RPC
Xymon mailing list Xymon at xymon.com http://lists.xymon.com/mailman/listinfo/xymon
On Tue, October 3, 2023 13:06, Tom Schmidt wrote:
J.C. and 4.4-alpha1 testers, I have done a little bit of testing of 4.4-alpha1 on Rocky Linux 8.8 (server and client) and on a busybox Linux install on a NAS. I looked into the gcc 8.5.0 compiler warnings that I got on Rocky Linux, and have corrected them, or silenced the ones that should not be an issue. Attached are two context diff patch files, one to fix the compiler warnings, and one to make a couple updates that I had applied to 4.3.30 previously. My updates fix the busybox Linux client build, and enhance the temperature graph to optionally display both Celsius and Fahrenheit readings.
I have not fully tested 4.4-alpha1 yet, but wanted to get these firstpatches released for 4.4-alpha2.
Tom Schmidt
Thanks for the patches!
Yes, the build errors had been a concern for me as well; determining which sections were protected by correct math and which weren't was going to take some time, and this is quite helpful.
I've created a 4.3.31 branch for similar concerns and many of these are just as applicable there. There's also a corruption bug on some loads that is crashing on F28 (and F29), so that release will probably just be build fixes for the stable tree.
Regards, -jc
Tom,
I realise that you're working with Rocky 8, but I just want to note that the lib/tcplib.c patch breaks compilation on RHEL7.
The function ASN1_STRING_get0_data() appears in openssl-1.1, but not in RHEL7 / openssl-1.0.
I found that there's an openssl version number in the openssl/opensslv.h include file, which allows the attached patch to select the correct function. With this modification to your patch, the compilation completes on RHEL7, RHEL8 and RHEL9.
Regards,
Ralph Mitchell
On Tue, Oct 3, 2023 at 11:12?PM J.C. Cleaver <cleaver at terabithia.org> wrote:
On Tue, October 3, 2023 13:06, Tom Schmidt wrote:
J.C. and 4.4-alpha1 testers, I have done a little bit of testing of 4.4-alpha1 on Rocky Linux 8.8 (server and client) and on a busybox Linux install on a NAS. I looked into the gcc 8.5.0 compiler warnings that I got on Rocky Linux, and have corrected them, or silenced the ones that should not be an issue. Attached are two context diff patch files, one to fix the compiler warnings, and one to make a couple updates that I had applied to 4.3.30 previously. My updates fix the busybox Linux client build, and enhance the temperature graph to optionally display both Celsius and Fahrenheit readings.
I have not fully tested 4.4-alpha1 yet, but wanted to get these firstpatches released for 4.4-alpha2.
Tom Schmidt
Thanks for the patches!
Yes, the build errors had been a concern for me as well; determining which sections were protected by correct math and which weren't was going to take some time, and this is quite helpful.
I've created a 4.3.31 branch for similar concerns and many of these are just as applicable there. There's also a corruption bug on some loads that is crashing on F28 (and F29), so that release will probably just be build fixes for the stable tree.
Regards, -jc
Xymon mailing list Xymon at xymon.com http://lists.xymon.com/mailman/listinfo/xymon
Hi JC,
At work I maintain a Xymon install with around 3000 hosts and 90000 checks. We have a lot of custom checks, some of which I want to share in the community in the future including a quite comprehensive Python client library.
One of the things we have is a set of automated checks that check the smart status of every drive we have. (Some hosts have 120 drives.) It generates a number of checks per host like ?smart-sda?, ?smart-sdb? and also a combo test called ?smarts?. I have a patch that I always apply to the Xymon source that allows me to use a regexp in the ?group-except? directive in hosts.cfg, so I can hide ?%smart-.*|othertest?, rather than having to specify ?smart-sda|smart-sdb|othertest? and any other drive that may appear.
I think this patch may be useful for more people than just me, so can we please include this?
Kind regards
Mark
PS: I am also happy to help with testing of new releases and features at home or at work on a larger data set.
Ralph, thanks for catching and fixing this for the OpenSSL version.
Tom Schmidt
On Tue, Oct 3, 2023 at 10:34?PM Ralph M <ralphmitchell at gmail.com> wrote:
Tom,
I realise that you're working with Rocky 8, but I just want to note that the lib/tcplib.c patch breaks compilation on RHEL7.
The function ASN1_STRING_get0_data() appears in openssl-1.1, but not in RHEL7 / openssl-1.0.
I found that there's an openssl version number in the openssl/opensslv.h include file, which allows the attached patch to select the correct function. With this modification to your patch, the compilation completes on RHEL7, RHEL8 and RHEL9.
Regards,
Ralph Mitchell
On Tue, Oct 3, 2023 at 11:12?PM J.C. Cleaver <cleaver at terabithia.org> wrote:
On Tue, October 3, 2023 13:06, Tom Schmidt wrote:
J.C. and 4.4-alpha1 testers, I have done a little bit of testing of 4.4-alpha1 on Rocky Linux 8.8 (server and client) and on a busybox Linux install on a NAS. I looked into the gcc 8.5.0 compiler warnings that I got on Rocky Linux, and have corrected them, or silenced the ones that should not be an issue. Attached are two context diff patch files, one to fix the compiler warnings, and one to make a couple updates that I had applied to 4.3.30 previously. My updates fix the busybox Linux client build, and enhance the temperature graph to optionally display both Celsius and Fahrenheit readings.
I have not fully tested 4.4-alpha1 yet, but wanted to get thesefirst
patches released for 4.4-alpha2.
Tom Schmidt
Thanks for the patches!
Yes, the build errors had been a concern for me as well; determining which sections were protected by correct math and which weren't was going to take some time, and this is quite helpful.
I've created a 4.3.31 branch for similar concerns and many of these are just as applicable there. There's also a corruption bug on some loads that is crashing on F28 (and F29), so that release will probably just be build fixes for the stable tree.
Regards, -jc
Xymon mailing list Xymon at xymon.com http://lists.xymon.com/mailman/listinfo/xymon
J.C., Sorry for the delay, but I have some updated patches for xymon 4.4 alpha1. I have compiled it on Rocky Linux 8 and 9 64-bit systems. The first set of patches corrects gcc warnings seen on gcc 8.5 (RL8) and 11.4.1 (RL9). I previously sent some patches for these when I tested on RL8 gcc 8.5, but have improved and/or corrected the patches based on RL9 gcc 11.4.1. The second set of patches includes some other bug fixes and enhancements, including:
- build/Makefile.Linux: Check for existence of replacement rpc
- build/Makefile.rules: Add XYMONRUNDIR to CFLAGS
- lib/compression.c: Handle LZ4 versions syntax change
- xymond/client/snmpcollect.c: Initialize nslsects
- xymond/etcfiles/tasks.cfg.DIST: Add IPv6 to admin-senders
- xymond/Makefile: correct missing variable substitutions
- xymond/webfiles/hostgraphs_header: remove XYMWEBHOST hyperlinks
- xymond/xymonfetch.c: Fix gcc warning
- xymond/wwwfiles/gifs/xymonbody.css: Add table td/th classes, tooltips
As a side note, I have also been trying to get a xymon 4.4a1 client to work on an Arm-based NAS system that I have, which the xymon 4.3.30 client is working fine on. I may have more patches to submit once I figure out how to fix it.
Thanks again! Tom Schmidt
On Wed, Oct 4, 2023 at 8:11?AM Tom Schmidt <tom at 4schmidts.com> wrote:
Ralph, thanks for catching and fixing this for the OpenSSL version.
Tom Schmidt
On Tue, Oct 3, 2023 at 10:34?PM Ralph M <ralphmitchell at gmail.com> wrote:
Tom,
I realise that you're working with Rocky 8, but I just want to note that the lib/tcplib.c patch breaks compilation on RHEL7.
The function ASN1_STRING_get0_data() appears in openssl-1.1, but not in RHEL7 / openssl-1.0.
I found that there's an openssl version number in the openssl/opensslv.h include file, which allows the attached patch to select the correct function. With this modification to your patch, the compilation completes on RHEL7, RHEL8 and RHEL9.
Regards,
Ralph Mitchell
On Tue, Oct 3, 2023 at 11:12?PM J.C. Cleaver <cleaver at terabithia.org> wrote:
On Tue, October 3, 2023 13:06, Tom Schmidt wrote:
J.C. and 4.4-alpha1 testers, I have done a little bit of testing of 4.4-alpha1 on Rocky Linux
8.8
(server and client) and on a busybox Linux install on a NAS. I looked into the gcc 8.5.0 compiler warnings that I got on Rocky Linux, and have corrected them, or silenced the ones that should not be an issue. Attached are two context diff patch files, one to fix the compiler warnings, and one to make a couple updates that I had applied to 4.3.30 previously. My updates fix the busybox Linux client build, and enhance the temperature graph to optionally display both Celsius and Fahrenheit readings.
I have not fully tested 4.4-alpha1 yet, but wanted to get thesefirst patches released for 4.4-alpha2.
Tom Schmidt
Thanks for the patches!
Yes, the build errors had been a concern for me as well; determining which sections were protected by correct math and which weren't was going to take some time, and this is quite helpful.
I've created a 4.3.31 branch for similar concerns and many of these are just as applicable there. There's also a corruption bug on some loads that is crashing on F28 (and F29), so that release will probably just be build fixes for the stable tree.
Regards, -jc
Xymon mailing list Xymon at xymon.com http://lists.xymon.com/mailman/listinfo/xymon
the procs check is the usual out-of-the-box every-five-minutes check done automagically by xymon of course.
If one was to update analysis.cfg on the xymon server is there a way to immediately force a procs check rather than wait several minutes for the check to be run?
didds
participants (5)
-
cleaver@terabithia.org
-
didds3@yahoo.co.uk
-
mark@stitson.com
-
ralphmitchell@gmail.com
-
tom@4schmidts.com