[GitHub] [pulsar-helm-chart] toneill818 commented on issue #39: Bug in podAntiAffinity labels

[GitBox <gi...@apache.org>]

toneill818 commented on issue #39:
URL: https://github.com/apache/pulsar-helm-chart/issues/39#issuecomment-672891558


   @rvashishth The bug is in the labelSelector for podAntiAffinity. 
   ```yaml
   affinity:
     podAntiAffinity:
       requiredDuringSchedulingIgnoredDuringExecution:
         - labelSelector:
           matchExpressions:
             - key: "app"
               operator: In
               values:
                 - "{{ template "pulsar.name" . }}-{{ .Values.zookeeper.component }}" # This is where the issue is, it will never match
   ```
   The app is set to pulsar.name-component. So it looks for app: pulsar-zookeeper, but the template for the labels sets the app to pulsar.name. Removing the "-component" should fix this. I can open a PR with this fix if you would like. Also with this change you will need to have as many nodes as the highest replicas for any of the stateful sets. Should we set the affinity to `preferredDuringSchedulingIgnoredDuringExecution`?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org