public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
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.


             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