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

Unable to install multiple Weaviate clusters in same namespace #115

Open
samos123 opened this issue Feb 9, 2023 · 2 comments
Open

Unable to install multiple Weaviate clusters in same namespace #115

samos123 opened this issue Feb 9, 2023 · 2 comments

Comments

@samos123
Copy link
Contributor

samos123 commented Feb 9, 2023

The templates hardcode a resource name instead of relying on the helm release name. So when you try to deploy another weaviate cluster it would fail because there is already a statefulset named weaviate.

I propose to instead use this format for all resources:

kind: StatefulSet     
metadata:     
  name: "{{ .Release.Name }}-weaviate"     
  labels:                                                                                                             
    name: "{{ .Release.Name }}-weaviate"                                                                          
    app: "{{ .Release.Name }}-weaviate"                                                                          
    app.kubernetes.io/name: "{{ .Release.Name }}-weaviate"                                                     
    app.kubernetes.io/managed-by: helm                         
    app.kubernetes.io/component: database                       
    app.kubernetes.io/version: "{{ .Values.image.tag }}"

Note this feature will only be considered if there is end-user need for this. Please upvote this issue if this would be needed for your use case and leave a comment with your use case.

@felixthekraut
Copy link

This would be quite helpful for our internal development environments so we can run different load test instances without having to muck around with additional namespaces. Can be useful for prod as well - for example very heavy work loads can be segregated to different clusters in the same namespace.

@pommedeterresautee
Copy link

pommedeterresautee commented Feb 9, 2024

Any update ? It is a bit surprising it is still not yet supported... seems to be a design error in the chart as it s quite common and useful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants