charts icon indicating copy to clipboard operation
charts copied to clipboard

add support for an external database

Open fibbs opened this issue 2 years ago • 3 comments

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 avatar Nov 26 '23 17:11 fibbs

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 avatar Nov 28 '23 22:11 gabe565

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

gabe565 avatar Dec 10 '23 22:12 gabe565