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/nextcloud: suggest poolConfig option #48094
Conversation
@@ -114,6 +114,21 @@ in { | |||
''; | |||
}; | |||
|
|||
poolConfig = mkOption { | |||
type = types.str; |
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.
I think this might be a types.lines
?
Note that you can use I can see how it might be useful to override the default completely though. |
Hi @eqyiel, thanks to review this pull request ! So I follow your guidelines. Is that ok for you ? |
@Vskilet It looks reasonable to me but I don't have commit rights to this repo, so you'll have to convince someone else 😅 |
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.
Cool, thanks! It's better to make the pm options overrideable like this so that the other options don't have to be repeated in this case.
@GrahamcOfBorg test nextcloud |
Success on aarch64-linux Attempted: tests.nextcloud No partial log is available. |
Success on x86_64-linux Attempted: tests.nextcloud No partial log is available. |
(cherry picked from commit 2dcd512)
(cherry picked from commit 2dcd512)
Motivation for this change
I suggest to add options for the poolConfig because on my server, default config is really slow and it's missing
pm.max_requests
Do you think it's usefull ?
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)