From 9683835f04a0c1784ab2e7667ff8bedd5b084da6 Mon Sep 17 00:00:00 2001 From: provokateurin Date: Wed, 8 Jan 2025 18:14:13 +0100 Subject: [PATCH 1/2] nixos/nextcloud: Update logreader warning description The error is only logged as an info 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/modules/services/web-apps/nextcloud.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/nixos/modules/services/web-apps/nextcloud.md b/nixos/modules/services/web-apps/nextcloud.md index 8d9aa878c2877..9d49af27f55c9 100644 --- a/nixos/modules/services/web-apps/nextcloud.md +++ b/nixos/modules/services/web-apps/nextcloud.md @@ -245,7 +245,7 @@ that are managed by Nix. If you want automatic updates it is recommended that yo ## 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} +### Logreader application only supports "file" log_type {#module-services-nextcloud-warning-logreader} This is because @@ -253,10 +253,10 @@ This is because * 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 +The logreader application doesn't work, as it was the case before. +So nothing actionable here by default. Alternatively you can -* disable the logreader application to shut up the "error". +* disable the logreader application to shut up the "info". 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. From 94307683c174357137ec99c2ea82b6ebe02c29a3 Mon Sep 17 00:00:00 2001 From: Maximilian Bosch Date: Fri, 14 Feb 2025 09:28:39 +0100 Subject: [PATCH 2/2] nixos/nextcloud: reword docs about logs a bit Originally, I only wanted to remove "The logreader application doesn't work, as it was the case before.". But then, the rest sounded a little weird, so I reworded the paragraph a bit more ;-) --- nixos/modules/services/web-apps/nextcloud.md | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/nixos/modules/services/web-apps/nextcloud.md b/nixos/modules/services/web-apps/nextcloud.md index 9d49af27f55c9..c9773f779ce7d 100644 --- a/nixos/modules/services/web-apps/nextcloud.md +++ b/nixos/modules/services/web-apps/nextcloud.md @@ -253,16 +253,12 @@ This is because * 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. -So nothing actionable here by default. Alternatively you can +If you want to view logs in the admin panel, +set [](#opt-services.nextcloud.settings.log_type) to "file". -* disable the logreader application to shut up the "info". - - 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. +If you prefer logs in the journal, disable the logreader application to shut up the +"info". 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. ## Maintainer information {#module-services-nextcloud-maintainer-info}