djjudas21 / charts

Collection of Helm charts
15 stars 6 forks source link

External Database support for LLDAP #78

Closed flashpixx closed 1 month ago

flashpixx commented 1 month ago

Is your feature request related to a problem ?

Yes, because I'm using the Helm chart for an centalized authentification service of another project, which ships also with Postgres, I didn't want to run multiple postgres instances

Describe the solution you'd like.

Extend the chart so that any kind of external databases can be used, e.g. multiple section externalPostgres or externalMariaDB could be used, if one of these is set, the settings will be used, with that my external database can be used

Describe alternatives you've considered.

NONE

Additional context.

No response

djjudas21 commented 1 month ago

Hey, thanks for your suggestion. Which helm chart is your request about? I'm assuming lldap if this is related to authentication?

flashpixx commented 1 month ago

yes it is the lldap chart, I have got an external Postgresql database so I have to pass the connection settings only to the chart and need to disable the built-in chart. I think the Bitnami MLFlow chart could be a nice draft for that https://artifacthub.io/packages/helm/bitnami/mlflow?modal=values&path=externalDatabase

djjudas21 commented 1 month ago

Thanks for clarifying. I agree it would be a useful feature. I'm away from work for 2 weeks but I'll have a look at this when I'm back. In the meantime I'm always happy to receive pull requests 🙂

flashpixx commented 1 month ago

I have tested it locally with my setup, but it seems that git on the pipeline breaks