Skip to content
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

Allow disabling the log database #6743

Closed
1 task done
onedr0p opened this issue Apr 19, 2024 · 2 comments · Fixed by #6765
Closed
1 task done

Allow disabling the log database #6743

onedr0p opened this issue Apr 19, 2024 · 2 comments · Fixed by #6765

Comments

@onedr0p
Copy link
Contributor

onedr0p commented Apr 19, 2024

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Is your feature request related to a problem? Please describe

I am using Sonarr with Postgres and it's a bit inconvenient to have to have 2 databases per instance of Sonarr. In my setup I have logs via stdout which are also going to a log aggregator (Loki), Sonarr also writes the logs to disk and writes logs to another database. I don't see the need to have 3 places for the logs.

Describe the solution you'd like

It would be awesome if there was a way in the config.xml to disable the log database completely and disable any feature in the app that relies on it (if any).

Describe alternatives you've considered

There really isn't any alternative.

Anything else?

Radarr ref: Radarr/Radarr#9110

@markus101
Copy link
Member

markus101 commented Apr 19, 2024

Not something we're going to commit to doing at this time, if someone is interested in picking this up we'll leave the issue tagged.

A couple other things that could be done:

  • Ability to set log level for DB logs in the config file
  • Optionally use SQLite DB for logs regardless, but this only prevents Postgres from dealing with them

@sillock1
Copy link
Contributor

I'd be happy to pick this up after this issue is done.

I did open a Radarr PR for toggling the log database. Is this something you'd be happy for me to bring into here?
Radarr/Radarr#9943

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants