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: Add services.nextcloud.autoUpdateApps #61717
Conversation
@maralorn Thanks for the PR! Could you add a paragraph to Maybe also enable it in the tests, so we hit the code paths there (even though we obviously can't test timers get fired) |
@flokli There you go! |
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.
Thanks for the followup!
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.
LGMT :-)
Can you squash this together?
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.
In fact, I missed that part - references only exist on leaf nixos options, not all nodes. Sorry for the noise!
nixos/nextcloud: Add documentation for nextcloud app installation and updates nixos/nextcloud: Enable autoUpdateApps in nextcloud test nixos/nextcloud: Fix typo in nixos/modules/services/web-apps/nextcloud.xml Co-Authored-By: Florian Klink <flokli@flokli.de> nixos/nextcloud: Escape html in option description nixos/nextcloud: Fix autoUpdateApps URL in documentation. Co-Authored-By: Florian Klink <flokli@flokli.de>
Thanks a lot! |
Can we backport this to 19.03? It's just a new option. That shouldn't break anything. |
backported to 19.03 in 246ebba. |
Motivation for this change
This change adds the options
services.nextcloud.autoUpdateApps.enable
andservices.nextcloud.autoUpdateApps.startAt
.If set it configures a systemd-timer unit to run
occ app:update --all
at the specified time.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)