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/rspamd: Remove non-working socket activation #47449
Conversation
SocketMode = mkIf (isUnixSocket each.value.socket) each.value.mode; | ||
}; | ||
}; | ||
}) allMappedSockets)); | ||
}; | ||
imports = [ | ||
(mkRenamedOptionModule [ "services" "rspamd" "bindSocket" ] [ "services" "rspamd" "workers" "normal" "bindSockets" ]) |
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.
Please also add it as mkRemovedOptionModule
here.
c7eaf25
to
1de5c28
Compare
@Mic92 it was bit late when I made this so I forgot the Should be fixed now. |
|
The socket activation I added to the rspamd module doesn't actually work and can't be made to work without changes to rspamd. See: NixOS#47421 See: rspamd/rspamd#2035
1de5c28
to
783a58f
Compare
@xeji I clearly shouldn't make PRs late at night. The test is now removed. |
Great. Let me run the remaining tests locally before merging. |
Tests run fine here. |
backported to 18.09 in 3108f74 |
Motivation for this change
The socket activation I added to the rspamd module doesn't actually work
and can't be made to work without changes to rspamd.
See: #47421
See: rspamd/rspamd#2035
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)