zabbix:1.8 -> 4.0 | remove old packages

[PsyanticY]

Motivation for this change

Things done

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
• Built on platform(s)
  • NixOS
  • macOS
  • other Linux distributions
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Assured whether relevant documentation is up to date
• Fits CONTRIBUTING.md.

---

[PsyanticY]

#40015

[PsyanticY]

@matthewbauer can you please have a look

[risicle]

nox-review builds for me, non-nixos linux x86_64

[AmineChikhaoui]

@GrahamcOfBorg build zabbix

[matthewbauer]

@GrahamcOfBorg build zabbix

[PsyanticY]

@matthewbauer thanks

[aanderse]

@PsyanticY @matthewbauer This breaks zabbix for anyone who was running it on NixOS as it entirely removes postgresql support. Please see #63844 for a more thorough implementation which doesn't irreparable break existing installs. Also note that you dropped versions of zabbix which are supported upstream while keeping versions of zabbix which are not.

[PsyanticY]

@aanderse your right about the possibility that dropping postgresql can break for people using it but there was no one using it since the module itself was broke ( notice the extra zabbix_server in the execstart in the systemd unit : serviceConfig.ExecStart = "@${pkgs.zabbix.server}/sbin/zabbix_server zabbix_server --config ${configFile}";) . for the 2.2 and previous version there end of support is august 31 (check this) . I did remove them since many people tried to upgrade zabbix but they got annoyed and dropped it since they needed to manage many packages, Also as a rule of thumb i think in nixpkgs we tend to only keep the latest version on any packages. For your PR i think it is way better then this one since it introduces many more functionalities and options

[aanderse]

That isn't broke. That is a systemd feature.

[aanderse]

Sorry I think I was confused and thought you dropped 3.0 which is supported, but didn't drop 3.4 which is not supported.

Also while some people argue only keeping latest version is the policy of NixOS this is not the case. NixOS has a flexible policy about keeping LTS releases. See php and postgtesql as examples where we provide our users long term stability, which is very important from my perspective.

[aanderse]

@PsyanticY To follow up why this service was not broken I'll quote https://www.freedesktop.org/software/systemd/man/systemd.service.html:

If the executable path is prefixed with "@", the second specified token will be passed as "argv[0]" to the executed process (instead of the actual filename), followed by the further arguments specified.

[PsyanticY]

@aanderse thanks for clearing that out i didn't knew about that.