[GitHub] [pulsar-helm-chart] sag-tjegorow opened a new pull request, #351: Configure custom topologyKey for podAntiAffinity

["sag-tjegorow (via GitHub)" <gi...@apache.org>]

sag-tjegorow opened a new pull request, #351:
URL: https://github.com/apache/pulsar-helm-chart/pull/351

   Adds the ability to configure anti-affinity topology key.
   
   ### Motivation
   
   Ability to set custom anti-affinity topology key should make it easier to use pulsar-helm-charts in a high availability multi-AZ cluster (we can use zone/region topology key). Also custom topology will allow the use of keys that are non-standard/not-well-known (see https://kubernetes.io/docs/reference/labels-annotations-taints/), the example being "topology.ebs.csi.aws.com/zone`" from EBS (see https://github.com/kubernetes-sigs/aws-ebs-csi-driver/issues/729), or just being deprecated (like "failure-domain.beta.kubernetes.io/zone") but still used on older k8s clusters.
   
   I realize there are already 2 PRs waiting for the acceptance that kind of do the same thing:
   * https://github.com/apache/pulsar-helm-chart/pull/61 -- at this moment it is a bit stale as it was created in September 2020 (and there are more other changes/refactorings).
   * https://github.com/apache/pulsar-helm-chart/pull/127 -- functionally very similar to this one (except soft affinity weight configuration) but does not allow to configure custom topology key. Being able to configure the topology key makes this PR simpler.
   
   ### Modifications
   
   Just the ability to configure topology keys using new values: <component>.affinity.anti_affinity_topology_key -- defaults to currently hard-coded "kubernetes.io/hostname".
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar-helm-chart] lhotari merged pull request #351: Configure custom topologyKey for podAntiAffinity

["lhotari (via GitHub)" <gi...@apache.org>]

lhotari merged PR #351:
URL: https://github.com/apache/pulsar-helm-chart/pull/351


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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