-
Notifications
You must be signed in to change notification settings - Fork 52
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
Enhancement: Include Redis and Postgres chart so it can be deployed though Authelia chart #89
Comments
This increases the complexity of this chart quite a lot is my feeling
And technically you could also add OpenLDAP as an option. I think what would be best is to provide more concrete examples |
The plan is to add an auxiliary chart that includes this chart as an example for anyone wanting to do a full stack. The alternative is making the stack parts optional. Some have suggested the later approach may be better but I'm not entirely convinced since no compelling reasons have been provided. |
Sorry to comment on this old thread 🙂 I'm looking at Authelia for the first time today and was surprised that the chart did not include postgres/redis as optional dependencies. It's an outlier in this regard, because almost every other chart I use does include its dependencies (and Helm describes itself as a package manager, which implies dependency resolution). A satisfactory option could be to provide an umbrella chart that manages the whole stack, but I definitely think you need to do more than simply provide examples, because otherwise it defies the point of a "package manager" if you have to manually install dependencies before installing Authelia. |
I thought it's fixed so this should be closed? |
The issue will be closed once the 0.9.0 version is merged, it's not released yet just available for testing. It'll automatically be closed when this occurs, sorry for confusion. |
Sorry, that's my fault. I assumed it would be closed when the PR was merged, rather than when the commits hit |
The thinking is that you could specify things like:
Basically that means that this chart would deployed redis too if it's enabled and use it.
Same with the storage backend.
The text was updated successfully, but these errors were encountered: