From: Robin Roevens <robin.roevens@disroot.org>
To: development@lists.ipfire.org
Subject: [PATCH 0/4] [V2] zabbix_agentd: new maintainer/summary
Date: Wed, 07 Apr 2021 22:44:52 +0200 [thread overview]
Message-ID: <20210407204455.450-1-robin.roevens@disroot.org> (raw)
[-- Attachment #1: Type: text/plain, Size: 2802 bytes --]
Since a new version of Zabbix Agent 5 LTS was released before the
previous patch-set was reviewed, I resubmit my patchset as a V2 updating
current zabbix_agentd 4.2.6 to 5.0.10 as opposed to v5.0.9 in my
previous submission.
The other 3 patches in the set remain unchanged.
For reference I'll include the summary again:
This set of patches does not only update the binaries (well,
the first patch only does that) but also fixes some things
that I see as problematic in previous version:
- /usr/lib/zabbix is created for users to drop custom agent
modules in, however that dir was removed and recreated
on update as it was not in the backup. I added it to the
backup and prevented deletion of the directory if it is
not empty upon uninstall, so user-added content would
not disapear when the package is removed.
- Sometimes a new version of the agent will introduce
new configuration parameters. In general the Zabbix Agent
config file(s) should remain compatible, but we never
know what the future will bring us; and the user may
miss out on new features introduced with new parameters
in the config file. However we don't want to plain overwrite
the configfile as the user may (probably has) have changed
it.
Currently on upgrade configfiles are backed up, removed,
new are installed, then overwritten by the old ones from
the backup. Ending with the old config and the new agent.
I didn't find an example of another package doing something
similar, so I chose to save the new configfile(s) as
.ipfirenew-files like RPM-based distro's do with
.rpmnew-files. If the original config file is absent
the install script will automatically strip the .ipfirenew
extension. And if the new config file does not differ from
the currently installed one, the .ipfirenew-file is removed.
The install-script will also issue warning messages if
such .ipfirenew-files are left on the filesystem, requesting
the user to manually investigate and possibly merge the
configfile. I hope those warnings are visible in the pakfire
output.
A side effect is that the config files are also not removed
when the package is uninstalled. I don't see a problem here
for the zabbix own config-files. But it may pose a risk
concerning the sudoers-file?
- I added a few IPFire specific monitoring items to the agent
config which can be used for more in-depth monitoring of
the IPFire installation. The user is of course free to
use my template available on share.zabbix.com or github
to monitor those items, or create their own template.
Thanks for considering this patch-set. Please be honest but gentle
commenting on it :-).
Regards
Robin
--
Dit bericht is gescanned op virussen en andere gevaarlijke
inhoud door MailScanner en lijkt schoon te zijn.
next reply other threads:[~2021-04-07 20:44 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-07 20:44 Robin Roevens [this message]
2021-04-07 20:44 ` [PATCH 1/4] [V2] zabbix_agentd: Update to v5.0.10 (LTS) Robin Roevens
2021-04-09 19:25 ` Adolf Belka
2021-04-10 21:05 ` Robin Roevens
2021-04-12 10:27 ` Michael Tremer
2021-04-12 11:23 ` Adolf Belka
2021-04-12 13:48 ` Michael Tremer
2021-04-12 10:26 ` Michael Tremer
2021-04-07 20:44 ` [PATCH 2/4] [V2] zabbix_agentd: Fix agent modules directory Robin Roevens
2021-04-09 19:36 ` Adolf Belka
2021-04-10 21:13 ` Robin Roevens
2021-04-12 10:26 ` Michael Tremer
2021-04-12 10:50 ` Robin Roevens
2021-04-12 10:52 ` Michael Tremer
2021-04-12 11:38 ` Robin Roevens
2021-04-12 13:45 ` Michael Tremer
2021-04-07 20:44 ` [PATCH 3/4] [V2] zabbix_agentd: Better configfile handling during update Robin Roevens
2021-04-07 20:44 ` [PATCH 4/4] [V2] zabbix_agentd: Add IPFire specific userparameters Robin Roevens
2021-04-12 10:36 ` Michael Tremer
2021-04-12 22:16 ` Robin Roevens
2021-04-15 11:21 ` Michael Tremer
2021-04-15 13:12 ` Robin Roevens
2021-04-15 20:34 ` Robin Roevens
2021-04-19 13:42 ` Michael Tremer
2021-04-19 13:37 ` Michael Tremer
2021-04-19 20:50 ` Robin Roevens
2021-04-12 10:32 ` [PATCH 0/4] [V2] zabbix_agentd: new maintainer/summary Michael Tremer
2021-04-12 21:19 ` Robin Roevens
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210407204455.450-1-robin.roevens@disroot.org \
--to=robin.roevens@disroot.org \
--cc=development@lists.ipfire.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox