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

add support for an external database #437

Open
fibbs opened this issue Nov 26, 2023 · 2 comments
Open

add support for an external database #437

fibbs opened this issue Nov 26, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@fibbs
Copy link

fibbs commented Nov 26, 2023

Chart Name

paperless-ngx

Is your feature request related to a problem? Please describe.

No response

Describe the solution you'd like

I like to use PGO operator for all Postgres databases as it offers out-of-the-box upgrades, database backups, point-in-time restores etc. I would really like to use a PGO managed database for paperless-ngx, so I miss the option to use an external database in this helm chart.
Ideally would be an additional structure in the values.yaml like this:

externalPostgresql:
  enabled: true
  host: ""
  userName: ""
  password: ""
  dbName: ""
  existingSecret:
    secretName: ""
    userNameSecretKey: ""
    passwordSecretKey: ""
    hostSecretKey: ""
    dbNameSecretKey: ""
    postgresUserSecretKey: ""

With this example one could define wether to use an external database or not, and then chose from either using the values from the host, userName, password and other values, or to refer to an EXISTING secret and the keys within this secret being used. If one key would not be defined to be used in the existing secret, the appropriate "direct" option would be used, then.

Would a PR for such a feature be well received?

Describe alternatives you've considered

No response

Additional context

No response

@fibbs fibbs added the enhancement New feature or request label Nov 26, 2023
@gabe565
Copy link
Owner

gabe565 commented Nov 28, 2023

At the moment, this is doable by setting the Paperless-ngx database envs. I wouldn't be opposed to adding Helm values if more people prefer configuring them this way, but I usually try to keep the number of config values to a minimum in these charts. That keeps things simpler, and since the bjw-s common chart supports using hardcoded envs, valueFrom references, or envFrom, it's has been configurable enough for my use.

What do you think?

@gabe565
Copy link
Owner

gabe565 commented Dec 10, 2023

Hmm I'm rethinking this. I think an externalDatabase config would make sense and shouldn't be too much trouble to add!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants