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/pdns-recursor: use upstream systemd unit #100892
Conversation
Sure! I have been running pdns-recursor for a few years (nothing fancy besides a custom forward zone), though I don't use powerdns or dnsdist. So, about the other PRs what you tried/tested is probably more than I can do but I'll try review to review them all. |
@ofborg eval |
@GrahamcOfBorg test pdns-recursor ncdns |
be00266
to
dc790c1
Compare
@rnhmjoj I think I have addressed all concerns. Let me know what you think. |
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.
The changes looks good now and my tests are passing. Seems good to go.
As always, thanks for your review @rnhmjoj 🎉 |
Motivation for this change
pdns-recursor
upstream offers really goodsystemd
units, compared to many other software vendors. We should use what they provide.@rnhmjoj I'm not sure if the third commit "declare module user as system user" is desired or not, but I thought that seems to be the way things are going since RFC 52 so I figured I would throw it in here. If this isn't desired feel free to mention and I can toss it.
I don't run
pdns-recursor
(ordnsdist
, orpowerdns
, for that matter) yet. I have started reviewing this software for potential use, but I'm not able to properly test these yet. I'll depend on you to adequately test this PR, if you don't mind.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)