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 mongo as dependancie in the chart #1

Open
Tchoupinax opened this issue Jul 5, 2020 · 4 comments
Open

Add mongo as dependancie in the chart #1

Tchoupinax opened this issue Jul 5, 2020 · 4 comments

Comments

@Tchoupinax
Copy link

Hello,

With helm we can declare dependancy. I would love that this chart automatically install a mongoDB database (can be configuring in values.yaml of course). [https://helm.sh/docs/helm/helm_dependency/]

I can make the pull request if you are interested :)

Thank you for your work !

@suda
Copy link
Owner

suda commented Jul 6, 2020

Hi! Yes, it is possible to declare a dependency but there are many ways of deploying MongoDB with Helm. There's the default Bitnami chart, Sharded MongoDB one and you even can use kubedb operator to install it. It's also possible to just use an external, hosted MongoDB. I didn't want to make decisions for anyone, especially that everyone can have different needs. What if, instead of depending on a particular MongoDB, we update the README to tell the users how to quickly deploy it as well?

@Tchoupinax
Copy link
Author

I agree, paragraph in the readme with one way to quickly deploy a MongoDB instance is a good thing.
Do you want i suggest it ?

@ghost
Copy link

ghost commented Dec 31, 2020

Those who want to preview Ackee locally using K3D or KiND would find value in a MongoDB integration. It doesn't have to be released with the software. You could add an examples directory and create some common integrations to help reduce friction for those evaluating the software.

Here's how Gitea does it:
https://gitea.com/gitea/helm-chart/src/commit/94ee162ad57b7fe43ec3934b19431784cb796ff7/values.yaml#L109-L116

Note users can turn on and off the DB so they can use their own external database. This is a familiar pattern with Helm charts.

@suda
Copy link
Owner

suda commented Jan 5, 2021

The Gitea approach might work! Anyone willing to create a PR with this? :)

@suda suda transferred this issue from suda/ackee-chart Jan 7, 2021
@ghost ghost mentioned this issue Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants