-
Notifications
You must be signed in to change notification settings - Fork 29
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 affinity to template #89
Comments
That's not currently possible. PRs are welcome though 🙂 We could add deploy:
affinity:
podAntiAffinity:
preferredDuringSchedulingIgnoredDuringExecution:
- podAffinityTerm:
labelSelector:
matchLabels:
clusterName: {{ include "infinispan-helm-charts.name" . }}
app: infinispan-pod
topologyKey: kubernetes.io/hostname
weight: 100 |
You mean it's me need to create PR ? |
The feature may get added in the future by someone else (thanks for raising the issue), but submitting a PR yourself will definitely speed things up and ensure the feature is added.
In order for us to support this there are two changes required:
|
If i right understand affinity:
podAntiAffinity:
preferredDuringSchedulingIgnoredDuringExecution:
- podAffinityTerm:
labelSelector:
matchLabels:
clusterName: {{ include "infinispan-helm-charts.name" . }}
app: infinispan-pod
topologyKey: kubernetes.io/hostname
weight: 100 add this to statefulset template {{- if .Values.deploy.affinity }}
affinity :
{{- with .Values.deploy.affinity }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- end }} And need to add in value.yml: affinity: {} |
That looks correct 👍 |
So should i do the PR ? for this ? |
If you're able to, it would be much appreciated! |
Hello! |
Hello!
Can you create affinity template for deployment ?
Because i don need hostname, i need zones
Is this possible ?
Thank you
The text was updated successfully, but these errors were encountered: