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

Make topology implementation in helm configurable #5322

Closed
aberfeldy opened this issue Oct 18, 2019 · 4 comments
Closed

Make topology implementation in helm configurable #5322

aberfeldy opened this issue Oct 18, 2019 · 4 comments

Comments

@aberfeldy
Copy link

Feature Description

Currently etcd is hardcoded as topo-implementation in the helm chart Line 102
It would be nice to have that determined by a value from values.yaml, since zookeeper and consul are supported as well.

Use Case(s)

Users wouldn't have to rewrite/fork the helm chart in order to use a different topo-implementation on their Kubernetes cluster.

@morgo
Copy link
Contributor

morgo commented Oct 21, 2019

Thank you for the feature request.

@yindia
Copy link
Contributor

yindia commented Feb 24, 2020

@morgo Would like to work on this. Can this be assigned to me?

@morgo
Copy link
Contributor

morgo commented Feb 24, 2020

Be aware that there is also work in #5703 to add Kubernetes as a topo. I expect that after this is added, it will eventually become the default as it simplifies installs.

@yindia yindia removed their assignment May 29, 2021
@ajm188
Copy link
Contributor

ajm188 commented Jun 24, 2022

closing due to helm deprecation #6439

@ajm188 ajm188 closed this as not planned Won't fix, can't repro, duplicate, stale Jun 24, 2022
systay pushed a commit that referenced this issue Jul 22, 2024
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

4 participants