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/soju: add admin socket option and sojuctl wrapper #258520
Conversation
43fbdf7
to
90b5a26
Compare
90b5a26
to
961ec39
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
961ec39
to
a736a17
Compare
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.
Looks good. I didn't test it on my deployment because I use a custom module for soju, but your tests pass. I think we should wait for 23.11 to release and we can merge it after that.
a736a17
to
d6e5c8e
Compare
d6e5c8e
to
68589b3
Compare
68589b3
to
4def374
Compare
CI failure is #301490, which is now fixed, but I don't think I have a way to retrigger job without pushing new commits. |
The config is a in an arbitrary nix store path, and is necessary to invoke sojuctl. Without the wrapper, you have to read the systemd unit or use a similar trick to find it.
For now, just try creating a user with sojuctl. Actually connecting an irc client is pretty annoying.
4def374
to
e9fed4b
Compare
Description of changes
Add a
adminSocket.enable
option to theservices.soju
module, a wrapper for thesojuctl
command pointed at the service config file, and some tests.Using
sojuctl
to create the first admin user is necessary when setting up asoju
server. Before these changes, doing this was pretty awkward. You have to do something like this:And then you need to find the config file path in the nix store by inspecting the systemd unit.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)