Skip to content

Commit

Permalink
nixos/nextcloud: Remove obsolete known logreader warning
Browse files Browse the repository at this point in the history
The error is no longer logged since nextcloud/logreader#1449.
This was a bug in the app, since the error is not fixable by the admin due to the logging configuration.
  • Loading branch information
provokateurin committed Jan 8, 2025
1 parent 615478d commit 3b31bee
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 24 deletions.
3 changes: 0 additions & 3 deletions nixos/doc/manual/redirects.json
Original file line number Diff line number Diff line change
Expand Up @@ -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"
],
Expand Down
21 changes: 0 additions & 21 deletions nixos/modules/services/web-apps/nextcloud.md
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 3b31bee

Please sign in to comment.