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
nixos/zabbix: overhaul package & module #63844
Conversation
77a1543
to
9cbb595
Compare
@PsyanticY Well I didn't notice #63581... Can you please review this and provide feedback? |
…proxy-sqlite, zabbix.server-mysql, zabbix.server-pgsql, zabbix.web: init at 3.0, 4.0, and 4.2
Sorry for missing this! Had not realized there were multiple zabbix prs, but this one certainly looks to handle things well. Feel free to revert the other PR (or base on top of it). |
@matthewbauer No problem, it happens. My fault as well. I had started this branch before the other was publish so when I checked it wasn't there... but then when I actually pushed and made a PR I didn't check again. Oops. This PR has now been based on top of master after the other PR was merged. As mentioned I have been running this for roughly a month now. Mostly compatible with the merged PR, just a few tiny naming differences. Care to merge? |
@aanderse, as @matthewbauer said this one handle thing way better then the other pr so please revert the other or base on top of it. |
@PsyanticY Thanks for your approval on this. I regret that our work clashed so I'll double check open PRs more than once next time I work on something like this! Already rebased and briefly tested so I'm going ahead and merging this. Thank you for your support and listing yourself as a maintainer. I hope to hear from you again on the topic of zabbix on NixOS 😄 |
Thanks, no problem, it is all for the grater nix good :p :) |
Motivation for this change
Team effort between @mmahut and myself to cleanup Zabbix in NixOS as the module & packages have become fairly unmaintained.
I've been running this in a reasonably sized environment for about a month now.
Things not yet done
Add a NixOS test which tests all of the server, proxy, and agent (@mmahut)testzabbixProxy
module in a real environment (@aanderse)I'm comfortable to go ahead and merge this without a NixOS test in good faith that @mmahut will eventually get the up and running. As for the
zabbixProxy
module it will eventually get tested in my environment before 19.09 so I can fix an issues which may come up before then. At this point mostly waiting for @PsyanticY to give a thumbs up, as they had done some of the same work this PR does (my fault, I really should have searched for any open PRs when I pushed this).Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)