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/mongodb: fix running initialScript without a set root password #99037
Conversation
b14d8ce
to
5161708
Compare
@bluescreen303 @offlinehacker @cstrahan Could you review this PR please? |
I think this is an obvious one which can't break anything so I would love it if somebody could look over it. https://docs.mongodb.com/manual/reference/program/mongo/#bin.mongo shows that |
55dc1e2
to
aa7848b
Compare
Could somebody please just check the diff as this removes inserting a null value which should not make things worse. I would really like to get this merged. |
Co-authored-by: Timo Kaufmann <timokau@zoho.com>
0f1ce7b
to
ecb0291
Compare
@timokau Thanks for that fix! I kept your co-authorship btw. |
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 to me, thanks! OfBorg (our CI) is currently down, I will merge this once its back up and the checks are green. Please remind me if I forget.
I am sorry that this sat around unnoticed for so long. We have a bit of a problem with our PR backlog. Things get missed sometimes. You could give marvin (experimental) a try in your future PRs. You could even try it here, it will automatically remind me about this PR in three days if you post a message containing
/marvin opt-in
/status awaiting_reviewer
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
Motivation for this change
Running an
initialScript
without having set aninitialRootPassword
failed becausemogo
tried to login to the server with a nonexistent username (root).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)