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
Zabbix-agent 3.4 #39925
Zabbix-agent 3.4 #39925
Conversation
@GrahamcOfBorg build zabbix34 |
Success on x86_64-linux (full log) Attempted: zabbix34 Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: zabbix34 Partial log (click to expand)
|
@@ -13002,6 +13002,7 @@ with pkgs; | |||
|
|||
zabbix20 = callPackage ../servers/monitoring/zabbix/2.0.nix { }; | |||
zabbix22 = callPackage ../servers/monitoring/zabbix/2.2.nix { }; | |||
zabbix34 = callPackage ../servers/monitoring/zabbix/3.4.nix { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you know if zabbix34 is backwards compatible? If so, I think we would want to make this the default "zabbix" attribute (the top "zabbix" is still at 1.8!)/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@matthewbauer Hi, thanks for taking a look into this. I know that zabbix server is compatible with zabbix agent with lower versions, (we actually have an Amazon linux zabbix server (3.4) communicating with nixos servers that has the agent version 2.2 (https://www.zabbix.com/documentation/3.4/manual/appendix/compatibility) . For the other way around i don't believe it is the case, and even if it is the case people strongly advise against it. Not sure if it is possible to upgrade the default Zabbix server and agent to 3.4 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok it's probably fine for now.
Success on aarch64-linux (full log) Attempted: zabbix34 Partial log (click to expand)
|
Motivation for this change
The latest Zabbix agent offered by Nixpkgs is currently 2.2 which is pretty old, the default package is even older (1.8) which include the agent and the server. I think it is time to move to a newer version of Zabbix. It would be better to merge this pull request thought ( #31123 ) as it include the version 3.4 of the agent and the server in the default.nix package instead of 1.8. If it is preferable to keep the default.nix the way it is, I think my PR can be usefull for people who want to use the 3.4 version of the agent.
When testing I used the following nix expression to make sure to use the 3.4 version instead of the default one:
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)