From 3b31beeee2161917cb772f0bc15609c9a2384b15 Mon Sep 17 00:00:00 2001 From: provokateurin Date: Wed, 8 Jan 2025 18:14:13 +0100 Subject: [PATCH] nixos/nextcloud: Remove obsolete known logreader warning The error is no longer logged since https://github.com/nextcloud/logreader/pull/1449. This was a bug in the app, since the error is not fixable by the admin due to the logging configuration. --- nixos/doc/manual/redirects.json | 3 --- nixos/modules/services/web-apps/nextcloud.md | 21 -------------------- 2 files changed, 24 deletions(-) diff --git a/nixos/doc/manual/redirects.json b/nixos/doc/manual/redirects.json index a7ebb49b62b8a..226f1b4b114d8 100644 --- a/nixos/doc/manual/redirects.json +++ b/nixos/doc/manual/redirects.json @@ -524,9 +524,6 @@ "module-services-nextcloud-known-warnings": [ "index.html#module-services-nextcloud-known-warnings" ], - "module-services-nextcloud-warning-logreader": [ - "index.html#module-services-nextcloud-warning-logreader" - ], "module-services-nextcloud-maintainer-info": [ "index.html#module-services-nextcloud-maintainer-info" ], diff --git a/nixos/modules/services/web-apps/nextcloud.md b/nixos/modules/services/web-apps/nextcloud.md index 8d9aa878c2877..cbeea338ff049 100644 --- a/nixos/modules/services/web-apps/nextcloud.md +++ b/nixos/modules/services/web-apps/nextcloud.md @@ -243,27 +243,6 @@ Alternatively, extra apps can also be declared with the [](#opt-services.nextclo When using this setting, apps can no longer be managed statefully because this can lead to Nextcloud updating apps that are managed by Nix. If you want automatic updates it is recommended that you use web interface to install apps. -## Known warnings {#module-services-nextcloud-known-warnings} - -### Failed to get an iterator for log entries: Logreader application only supports "file" log_type {#module-services-nextcloud-warning-logreader} - -This is because - -* our module writes logs into the journal (`journalctl -t Nextcloud`) -* the Logreader application that allows reading logs in the admin panel is enabled - by default and requires logs written to a file. - -The logreader application doesn't work, as it was the case before. The only change is that -it complains loudly now. So nothing actionable here by default. Alternatively you can - -* disable the logreader application to shut up the "error". - - We can't really do that by default since whether apps are enabled/disabled is part - of the application's state and tracked inside the database. - -* set [](#opt-services.nextcloud.settings.log_type) to "file" to be able to view logs - from the admin panel. - ## Maintainer information {#module-services-nextcloud-maintainer-info} As stated in the previous paragraph, we must provide a clean upgrade-path for Nextcloud